Skip to content

Commit

Permalink
got rid of old tests
Browse files Browse the repository at this point in the history
  • Loading branch information
sonachitchyan committed Aug 9, 2023
1 parent 11c8a31 commit e6014cf
Showing 1 changed file with 0 additions and 27 deletions.
27 changes: 0 additions & 27 deletions tardis/transport/tests/test_doppler_factor.py
Original file line number Diff line number Diff line change
Expand Up @@ -189,30 +189,3 @@ def test_get_inverse_doppler_factor_full_relativity(mu, beta, expected):
mu, beta
)
assert_almost_equal(obtained, expected)


# @pytest.mark.parametrize(["mu", "r", "inv_t_exp"], [(-0.3, 5, 1e10)])
# def test_unphysical_doppler_factor(mu, r, inv_t_exp):
# # Set the params from test cases here
# # TODO: add relativity tests
# time_explosion = 1 / inv_t_exp

# # Perform any other setups just before this, they can be additional calls
# # to other methods or introduction of some temporary variables
# with pytest.raises(r_packet.SuperluminalError):
# obtained = frame_transformations.get_doppler_factor(
# r, mu, time_explosion
# )


# @pytest.mark.parametrize(["mu", "r", "inv_t_exp"], [(-0.3, 5, 1e10)])
# def test_unphysical_inverse_doppler_factor(mu, r, inv_t_exp):
# # Set the params from test cases here
# # TODO: add relativity tests
# time_explosion = 1 / inv_t_exp

# # Perform any other setups just before this, they can be additional calls
# # to other methods or introduction of some temporary variables
# with pytest.raises(r_packet.SuperluminalError):
# obtained = r_packet.get_inverse_doppler_factor(r, mu, time_explosion)
# Test here was written before switching to numba. Needs to be changed in the future

0 comments on commit e6014cf

Please sign in to comment.