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.

Sign up for GitHub

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

Jump to bottom

Check integrity of downloaded data files and avoid running dependent tests if a broken file is found. #41

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

Check integrity of downloaded data files and avoid running dependent tests if a broken file is found. #41

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
633d0a4
Check integrity of downloaded data files with md5 checksum.
shinji-s Aug 18, 2020
fb743a6
Merge branch 'develop' into download_by_fixtures
shinji-s Aug 18, 2020
b06df5e
Add description about download timeouts and resulting file corruption.
shinji-s Aug 18, 2020
46c3e99
Merge branch 'download_by_fixtures' of github.com:shinji-s/eccodes in…
shinji-s Aug 18, 2020
08dfe42
Clone ecbuild (and eccodes) repository preserving LF line endings so …
shinji-s Aug 18, 2020
f0fdf82
Make sure .md5 files have LF at the line ending on Windows as well be…
shinji-s Aug 19, 2020
4c8cce0
Fix fixture name references for download _gts.
shinji-s Aug 19, 2020
1a66d69
Remove unnecessary tweaking of core.autocrlf parameter.
shinji-s Aug 19, 2020
bf4ef79
Merge branch 'develop' into download_by_fixtures
shinji-s Sep 3, 2020
a98cd32
Merge branch 'develop' into download_by_fixtures
shinji-s Nov 10, 2020
7a38944
Removed a local patch from appveyor test because the fix had been mer…
shinji-s Nov 10, 2020
de8f819
Restore fixture-depenency definition that was lost during merge-confl…
shinji-s Nov 10, 2020
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
Prev Previous commit
Next Next commit
Merge branch 'develop' into download_by_fixtures
  • Loading branch information
@shinji-s
shinji-s committed Nov 10, 2020
commit a98cd32a149e2ee4083287b72834e08aa15ef210
29 changes: 19 additions & 10 deletions examples/python/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -136,21 +136,30 @@ if( HAVE_PYTHON AND ENABLE_EXTRA_TESTS AND HAVE_BUILD_TOOLS )
ecbuild_add_test( TARGET eccodes_p_bufr_encode_flight_test
TYPE SCRIPT
DEPENDS ${ptools}
COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/bufr_encode_flight.sh
COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/${test}.sh
CONDITION HAVE_PYTHON2 AND ENABLE_EXTRA_TESTS
ENVIRONMENT PYTHON=${PYTHON_EXECUTABLE}
RESOURCES flight_data.csv
)
set_tests_properties( eccodes_p_bufr_encode_flight_test PROPERTIES FIXTURES_REQUIRED fixture_download_bufrs )
endif()

# Add test which requires input CSV file
ecbuild_add_test( TARGET eccodes_p_bufr_encode_flight_test
TYPE SCRIPT
DEPENDS ${ptools}
COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/bufr_encode_flight.sh
CONDITION HAVE_PYTHON2 AND ENABLE_EXTRA_TESTS AND HAVE_BUILD_TOOLS
ENVIRONMENT PYTHON=${PYTHON_EXECUTABLE}
RESOURCES flight_data.csv
TEST_DEPENDS eccodes_download_bufrs )

# Conditional tests
if( HAVE_PYTHON AND HAVE_AEC AND ENABLE_EXTRA_TESTS )
ecbuild_add_test( TARGET eccodes_p_grib_ccsds_test
TYPE SCRIPT
DEPENDS ${ptools}
COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/grib_ccsds.sh
ENVIRONMENT PYTHON=${PYTHON_EXECUTABLE}
)
set_tests_properties( eccodes_p_grib_ccsds_test PROPERTIES FIXTURES_REQUIRED fixture_download_gribs )
endif()
ecbuild_add_test( TARGET eccodes_p_grib_ccsds_test
TYPE SCRIPT
DEPENDS ${ptools}
COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/grib_ccsds.sh
CONDITION HAVE_PYTHON2 AND HAVE_AEC AND ENABLE_EXTRA_TESTS
ENVIRONMENT PYTHON=${PYTHON_EXECUTABLE}
TEST_DEPENDS eccodes_download_gribs )

You are viewing a condensed version of this merge commit. You can view the full changes here.