Add more datasets to IASI L2 reader

[pnuu]

This PR adds datasets from /INFO/ and /Maps/ groups to the IASI L2 reader.

Also the IASI L2 tests were converted to pytest.

• Tests added

[codecov]

Codecov Report

Attention: Patch coverage is 93.46405% with 20 lines in your changes missing coverage. Please review.

Project coverage is 96.15%. Comparing base (91e758a) to head (cc64efc).
Report is 49 commits behind head on main.

Files with missing lines	Patch %	Lines
satpy/readers/hrit_base.py	57.14%	9 Missing ⚠️
satpy/readers/netcdf_utils.py	33.33%	8 Missing ⚠️
satpy/tests/reader_tests/test_iasi_l2.py	98.38%	2 Missing ⚠️
satpy/readers/iasi_l2.py	96.87%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3059      +/-   ##
==========================================
+ Coverage   96.11%   96.15%   +0.03%     
==========================================
  Files         383      383              
  Lines       55673    55822     +149     
==========================================
+ Hits        53511    53674     +163     
+ Misses       2162     2148      -14     

Flag	Coverage Δ
behaviourtests	3.88% <0.98%> (-0.01%)	⬇️
unittests	96.24% <93.46%> (+0.03%)	⬆️

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.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[pnuu]

Seems like bilinear resampling doesn't work with the current ("y", "x", "level") dimension order. If I change this to ("bands", "y", "x"). The nearest works also with the current ordering and naming, so the problem is within the bilinear resampler.

[djhoese]

My one small request that is not necessary would be rewriting most if not all of the test setup as a pytest fixture. You could have one that just produces the filename (Path object?) and is module scoped. Then have one that produces a Scene and one that produces a file handler (replaces self.reader) that are test scoped. That way any caching inside the Scene or file handler are not preserved between tests, but the test file is only created once.

[mraspaud]

LGTM

[pnuu]

I've now removed the test class and replaced the setup_method with fixtures. I tried adding scope="module" to test_data fixture, but Pytest complained with You tried to access the function scoped fixture tmp_path with a module scoped request object. so just removed the scoping and the tests now work (again).

[pnuu]

Ok, found the scope="module" fix from the other PR, a added that in 4053955

[coveralls]

Pull Request Test Coverage Report for Build 13989982277

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 153 of 156 (98.08%) changed or added relevant lines in 2 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.003%) to 96.268%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
satpy/readers/iasi_l2.py	31	32	96.88%
satpy/tests/reader_tests/test_iasi_l2.py	122	124	98.39%

Totals
Change from base Build 13989672119:	0.003%
Covered Lines:	54175
Relevant Lines:	56275

---

💛 - Coveralls

[mraspaud]

LGTM