Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added comment #763

Closed
wants to merge 12 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 30 additions & 2 deletions .github/workflows/developer.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
strategy:
fail-fast: true
matrix:
config: ["asan/warning check", "docs/code coverage"]
config: ["asan/warning check", "docs/code coverage", "extra"]

steps:

Expand All @@ -53,12 +53,21 @@ jobs:
with:
path: g2

- name: cache-data-extra
if: matrix.config == 'extra'
id: cache-data-extra
uses: actions/cache@v4
with:
path: ~/data_extra
key: data-developer-7

- name: cache-data
if: matrix.config == 'asan/warning check' || matrix.config == 'docs/code coverage'
id: cache-data
uses: actions/cache@v4
with:
path: ~/data
key: data-developer-5
key: data-developer-6

- name: asan
if: matrix.config == 'asan/warning check'
Expand All @@ -77,19 +86,38 @@ jobs:
set -x
cd g2
mkdir build
ls -l $GITHUB_WORKSPACE/nceplibs/NCEPLIBS-g2c/bin
doxygen --version
cd build
cmake -DBUILD_UTILS=ON -DG2C_COMPARE=OFF -DUSE_AEC=ON -DFTP_LARGE_TEST_FILES=ON -DENABLE_DOCS=ON -DCMAKE_PREFIX_PATH="$GITHUB_WORKSPACE/nceplibs/jasper;$GITHUB_WORKSPACE/nceplibs/NCEPLIBS-bacio;$GITHUB_WORKSPACE/nceplibs/NCEPLIBS-w3emc;$GITHUB_WORKSPACE/nceplibs/NCEPLIBS-ip;$GITHUB_WORKSPACE/nceplibs/NCEPLIBS-g2c" -DFTP_TEST_FILES=ON -DTEST_FILE_DIR=/home/runner/data ..
make -j2 VERBOSE=1
ctest --verbose --output-on-failure --rerun-failed
gcovr --root .. -v --html-details --exclude ../tests --exclude CMakeFiles --print-summary -o test-coverage.html &> /dev/null

- name: build
if: matrix.config == 'extra'
run: |
set -x
cd g2
mkdir build
ls -l $GITHUB_WORKSPACE/nceplibs/NCEPLIBS-g2c/bin
cd build
cmake -DBUILD_UTILS=ON -DG2C_COMPARE=ON -DUSE_AEC=ON -DFTP_TEST_FILES=ON -DTEST_FILE_DIR=/home/runner/data_extra -DFTP_LARGE_TEST_FILES=ON -DFTP_EXTRA_TEST_FILES=ON -DCMAKE_PREFIX_PATH="$GITHUB_WORKSPACE/nceplibs/jasper;$GITHUB_WORKSPACE/nceplibs/NCEPLIBS-bacio;$GITHUB_WORKSPACE/nceplibs/NCEPLIBS-w3emc;$GITHUB_WORKSPACE/nceplibs/NCEPLIBS-ip;$GITHUB_WORKSPACE/nceplibs/NCEPLIBS-g2c" ..
make -j2 VERBOSE=1
ctest --verbose --output-on-failure --rerun-failed

- name: cache-data
if: steps.cache-data.outputs.cache-hit != 'true'
run: |
mkdir ~/data
cp $GITHUB_WORKSPACE/g2/build/tests/data/* ~/data

- name: cache-data-extra
if: steps.cache-data-extra.outputs.cache-hit != 'true'
run: |
mkdir ~/data_extra
cp $GITHUB_WORKSPACE/g2/build/tests/data/* ~/data_extra

- name: upload-test-coverage
if: matrix.config == 'docs/code coverage'
uses: actions/upload-artifact@v4
Expand Down
5 changes: 1 addition & 4 deletions test_utils/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -79,9 +79,6 @@ endif()

# Run these shell tests.
if(BUILD_4)
if(BUILD_D)
gu_test(run_copygb2_tests2)
endif()
gu_test(run_cnvgrib_tests)
gu_test(run_degrib2_tests)
gu_test(run_grb2index_tests)
Expand All @@ -94,7 +91,7 @@ gu_test(run_copygb_tests)
gu_test(run_grbindex_tests)

if(G2C_COMPARE)
find_program(G2C_COMPARE g2c_compare)
find_program(G2C_COMPARE g2c_compare REQUIRED)
gu_test(run_copygb2_tests)
else()
message(STATUS "g2c_compare not found.")
Expand Down
3 changes: 3 additions & 0 deletions test_utils/run_tocgrib2super_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ export FORT11="../tests/data/rrfs.t12z.prslevfaa.f010.na3km.grib2"
export FORT51=out.grib2
../utils/tocgrib2super < data/tocgrib2_bad.nml && exit 1

# According to George V.: "if fortran unit 11 is, 40999888 bytes long,
# then fortran unit 12 should be a single 8 character text string
# containing this integer value."
echo "*** Running tocgrib2 with input/output defined. It will succeed."
ls -l ../tests/data/rrfs.t12z.prslevfaa.f010.na3km.grib2
export FORT11="../tests/data/rrfs.t12z.prslevfaa.f010.na3km.grib2"
Expand Down
Loading