fix: Fix directive naming Update check_nightly.py

[annwag]

The comment above the EXPENSIVE_DIRECTIVE variable indicates that the script is supposed to look for tests annotated with expensive_tests, but the code currently uses ultra_slow_test_.

This mismatch could lead to unintended behavior. I've updated the variable to correctly reflect the intended directive:

EXPENSIVE_DIRECTIVE = 'expensive_tests'  

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.66%. Comparing base (69ba684) to head (77ea270).

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #12729      +/-   ##
==========================================
- Coverage   70.66%   70.66%   -0.01%     
==========================================
  Files         848      848              
  Lines      173871   173871              
  Branches   173871   173871              
==========================================
- Hits       122874   122865       -9     
- Misses      45858    45865       +7     
- Partials     5139     5141       +2     

Flag	Coverage Δ
backward-compatibility	0.16% <ø> (ø)
db-migration	0.16% <ø> (ø)
genesis-check	1.35% <ø> (ø)
linux	69.22% <ø> (-0.02%)	⬇️
linux-nightly	70.24% <ø> (-0.01%)	⬇️
pytests	1.65% <ø> (ø)
sanity-checks	1.46% <ø> (ø)
unittests	70.49% <ø> (-0.01%)	⬇️
upgradability	0.20% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Longarithm]

1. the change is incorrect
2. looks like the variable is not even used... @nagisa ?

[nagisa]

Ah yeah, that's my bad -- just remove that variable definition entirely since we aren't checking for #[...] anymore like we used to. Other than that ultra_slow_test_ is how we annotate our expensive tests that run in nayduck today.