Skip to content

Commit

Permalink
Add a regression for issue #654
Browse files Browse the repository at this point in the history
  • Loading branch information
bcoconni committed Jun 18, 2022
1 parent 5fa0a2f commit 617fc27
Show file tree
Hide file tree
Showing 2 changed files with 45 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@ set(PYTHON_TESTS ResetOutputFiles
TestMagnetometer
TestLinearization
TestLinearActuator
TestPlanet)
TestPlanet
TestLighterThanAir)

foreach(test ${PYTHON_TESTS})
add_test(NAME ${test}
Expand Down
43 changes: 43 additions & 0 deletions tests/TestLighterThanAir.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
# TestLighterThanAir.py
#
# Tests that check that lighter than air vehicles (balloons, blimps, etc.) are
# working as expected.
#
# Copyright (c) 2022 Bertrand Coconnier
#
# This program is free software; you can redistribute it and/or modify it under
# the terms of the GNU General Public License as published by the Free Software
# Foundation; either version 3 of the License, or (at your option) any later
# version.
#
# This program is distributed in the hope that it will be useful, but WITHOUT
# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
# FOR A PARTICULAR PURPOSE. See the GNU General Public License for more
# details.
#
# You should have received a copy of the GNU General Public License along with
# this program; if not, see <http://www.gnu.org/licenses/>
#
from JSBSim_utils import JSBSimTestCase, RunTest, ExecuteUntil

import fpectl


class TestLighterThanAir(JSBSimTestCase):
def testValve(self):
fpectl.turnon_sigfpe()

fdm = self.create_fdm()
fdm.load_script(
self.sandbox.path_to_jsbsim_file("scripts", "weather-balloon.xml")
)

fdm.run_ic()
fdm["buoyant_forces/gas-cell/burst"] = 1.0

ExecuteUntil(fdm, 10.0)

fpectl.turnoff_sigfpe()


RunTest(TestLighterThanAir)

1 comment on commit 617fc27

@bcoconni
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I meant "Add a regression test" 😄 Of course, I didn't mean to introduce a regression on purpose in JSBSim

Please # to comment.