Skip to content

Commit

Permalink
test(transformer/replace-global-defines): remove panicking test (#7838)
Browse files Browse the repository at this point in the history
Follow-on after #7811. Remove the panicking test. It *does* currently
cause a panic if replacement contains scopes (e.g. `() => 123`) but
that's a bug. So we shouldn't have a test saying that it *should* panic.
  • Loading branch information
overlookmotel authored Dec 13, 2024
1 parent e812ae3 commit d72c888
Showing 1 changed file with 0 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -283,10 +283,3 @@ log(__MEMBER__);
let snapshot = visualizer.into_visualizer_text();
insta::assert_snapshot!("test_sourcemap", snapshot);
}

#[test]
#[should_panic(expected = "")]
fn test_complex() {
let config = ReplaceGlobalDefinesConfig::new(&[("__DEF__", "((() => {})())")]).unwrap();
test("__DEF__", "1", config.clone());
}

0 comments on commit d72c888

Please sign in to comment.