Update If-check to support CI/CD

[manushak]

Types of changes

• Enhancement (project structure, spelling, grammar, formatting)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

A description of the changes proposed in the Pull Request

• Add error summary log when checking directory files
• add process.exit(1) at the end of the catch

[jmcook1186]

sgtm!

[narekhovhannisyan]

@manushak @jmcook1186 it works but we have a bug and there is suggestion for improvement.

Right now we are getting the following:

Checking...

copy.yaml
if-check could not verify copy.yaml. The re-executed file does not match the original.

Files do not match!
tree.children.child-1.outputs.0.mock
source: exists
target: missing

generics.yaml
if-check successfully verified generics.yaml

instance-metadata.yaml
if-check successfully verified instance-metadata.yaml
copy.yaml
Files do not match!


Summary of all failing files:
2 of 3 files are passed.

tree.children.child-1.outputs.0.mock
source: exists
target: missing

Which means after summary we are getting logs

[narekhovhannisyan]

As an improvement to UI we can modify messages like this:

Checking...

Executing `copy.yaml`
✖ if-check could not verify copy.yaml. The re-executed file does not match the original.
Files do not match!
tree.children.child-1.outputs.0.mock
source: exists
target: missing

Executing `generics.yaml`
✔ if-check successfully verified `generics.yaml`

Executing `instance-metadata.yaml`
✔ if-check successfully verified `instance-metadata.yaml`

Executing `copy.yaml`
✖ Files do not match!
tree.children.child-1.outputs.0.mock
source: exists
target: missing

Tests summary:
2 of 3 files are passed.

[narekhovhannisyan]

Please check comments above

[jmcook1186]

ok I like the proposed updates but also liked that @manushak 's version separated out the individual emitted logs from the summary of failures. think we need some kind of break between the emitted messages and the "summary" - even if it is just a line break and summary title, e.g.

Checking...

Executing `copy.yaml`
✖ if-check could not verify copy.yaml. The re-executed file does not match the original.
Files do not match!
tree.children.child-1.outputs.0.mock
source: exists
target: missing

Executing `generics.yaml`
✔ if-check successfully verified `generics.yaml`

Executing `instance-metadata.yaml`
✔ if-check successfully verified `instance-metadata.yaml`

---
if-check verification failures:
---

Executing `copy.yaml`
✖ Files do not match!
tree.children.child-1.outputs.0.mock
source: exists
target: missing

---
Tests summary:
2 of 3 files are passed.

[narekhovhannisyan]

@jmcook1186 it doesn't look like an intentional solution since the title is before the summary and the rest is after. Yeah the end solution looks good