Green-Software-Foundation / if

Impact Framework
https://if.greensoftware.foundation/
MIT License
143 stars 40 forks source link

Create output files for automated testing #800

Closed jmcook1186 closed 2 months ago

jmcook1186 commented 3 months ago

What

Create a folder of output files that can be used for automated testing using if-check

Why

We currently have a set of manifests that can be used for manual testing. But we will soon have if-check available that will be used for automated tests in our CI/CD.

In order to do this, we need to reorganize our manifests so that:

@MariamKhalatova I opened a branch (reorg-manifest-folder) where I moved our manifest around into the following folders:

manifests
  |- examples
  |        |-builtins
  |        |-pipelines
  |
  |-outputs
  |        |-bugs
  |        |-features
  |        |-pipelines
  |        |-plugins

Now what we need to do is execute all the manifests. For each manifest in examples we just want to confirm that the examples execute successfully -if so, move on, if not, fix them. For each manifest in outputs we want to overwrite the manifest with its post-execution output file.

Context This will help support automated testing and supporting users with examples and sharing pipelines

Prerequisites/resources no

SoW (scope of work)

Acceptance criteria

zanete commented 3 months ago

@jmcook1186 and @MariamKhalatova to sync up and work on this together as @jmcook1186 would like to decide the file structure before any work is done

zanete commented 2 months ago

@jmcook1186 can you confirm that there are output files missing as we discussed in the planning and provide some details as to what should be done? 🙏

zanete commented 2 months ago

@narekhovhannisyan and @MariamKhalatova

EDIT: this should be done just before #812

zanete commented 2 months ago

@MariamKhalatova please go ahead with the audit 🙏

zanete commented 2 months ago

the missing outputs are failing, currently discussing the solution on slack

zanete commented 2 months ago

This is still blocked by the aggregation method

zanete commented 2 months ago

@manushak @narekhovhannisyan please review the PR for the missing manifests 🙏