getCurrentTestName shows wrong description

[erwindemoel]

Hi, I tried to implement your solution because we want to console.log the name of each running test within the fit(''). Here is my example code:

fit('testing my description', () => { console.log(JasmineSmacker.getCurrentTestName()); });

After following your steps, I do get a console.log with the name of one of our tests, however it is the wrong one. It logs the name of another test instead of the current one. How can I fix this so it logs the correct one?

[Jezorko]

Hey @erwindemoel, thanks for reporting this!
I've pushed a fix and will merge once the CI finishes (:
#10

[Jezorko]

The change is here:

smack-my-jasmine-up/src/main/JasmineSmacker.js

Lines 6 to 9 in c94c6ed

	[
	{testMethodToEnhance: it, setter: (newValue) => it = newValue},
	{testMethodToEnhance: fit, setter: (newValue) => fit = newValue}
	]

To get this thing working, I am globally replacing the it function. Now both it and fit are replaced.

[Jezorko]

Also I see that the discussion under the issue has been updated with a much nicer solution.