PecanProject / pecan-status-board

workflow integration testing status board
Other
2 stars 3 forks source link

Add Missing `curl` Package to `manual-test.yaml` as Dependency #14

Closed Sweetdevil144 closed 6 months ago

Sweetdevil144 commented 7 months ago

Summary

This pull request addresses an issue in our GitHub Actions workflow where the R environment was missing the curl package, leading to a failure in the execution of manual-test.yaml.

Issue Description

During the execution of our GitHub Actions workflow, particularly in the "Generate data" step within run-test-list.yaml, we encountered an error indicating that the curl package was not installed. The error message was as follows:

Error: Error in loadNamespace(x) : there is no package called ‘curl’

This issue was causing the workflow to halt prematurely, preventing the completion of our automated test suite.

Proposed Changes

To resolve this issue, I have added an additional command in the "Install dependencies" step of our manual-test.yaml workflow. This command explicitly installs the curl package in our R environment, ensuring that all necessary dependencies are available for our scripts.

The updated section of the workflow file is as follows:

- name: Install dependencies
  run: |
    remotes::install_deps(dependencies = TRUE)
    install.packages("XML")
    install.packages("devtools")
    devtools::install_github("PecanProject/rpecanapi")
    install.packages("dplyr")
    install.packages("tidyr")
    install.packages("curl") # Added to ensure the availability of the curl package
  shell: Rscript {0}

Impact

With the addition of the curl package, our automated testing workflow should now run without encountering the previous error. This will enable us to reliably generate and test data as part of our continuous integration process, ensuring the stability and reliability of our codebase.

Testing

After making this change, the workflow should be re-run to verify that the "Generate data" step completes successfully without encountering the previous error. Taking into account the amount of errors we have been seeing in our *.csv files, I'm sure we'll get to encounter another errors after this.

@infotroph Requesting a review on this pull request.