Skip to content

Conversation

@galanCA
Copy link

@galanCA galanCA commented Jan 27, 2023

Description

Adds CI test were it runs rake and uploads the report folder with all the standard tests performed by CSE.

Changes

  • Adds Test.yml which contains the test for the CI
  • Adds fail exit in rake file.
  • Changes the way modelkit because modelkit only allows inputs for terminal but github actions runs as a headless session therefore the inputs needed to be pipe into modelkit.

Test

  • Download modelkit caboodle
  • Install modelkit caboodle and waits for the installation process to be over
  • Runs rike for both section-5 test and weather-drivers test
  • Upload artifacts

@galanCA galanCA requested a review from nealkruis February 23, 2023 00:13
@galanCA
Copy link
Author

galanCA commented Feb 23, 2023

@nealkruis should we change the name of Test.yml to generate-test.yml?

@galanCA galanCA self-assigned this Feb 23, 2023
if !(FileUtils.uptodate?(target, src))
puts "\ncomposing...\n\n"
success = system(%Q|modelkit template-compose -f "#{c}" -o "#{output_dir + '/in.cse'}" base-#{tests}.pxt|)
success = system(%Q| echo base-#{tests}.pxt \| modelkit template-compose -f "#{c}" -o "#{output_dir + '/in.cse'}" |)
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change was made to allow passing the template file to modelkit.

@galanCA
Copy link
Author

galanCA commented Feb 28, 2023

@nealkruis It is ready to be reviewed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants