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.

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

Unit Tests for Cloudtrail/Support modules in Baselines #7307

Closed
3 tasks done
richgreen-moj opened this issue Jun 24, 2024 · 7 comments
Closed
3 tasks done

Unit Tests for Cloudtrail/Support modules in Baselines #7307

richgreen-moj opened this issue Jun 24, 2024 · 7 comments
Assignees

Comments

@richgreen-moj
Copy link
Contributor

richgreen-moj commented Jun 24, 2024

User Story

As a MP Engineer
I need to write some unit tests for the Cloudtrail and Support sub-modules sin the Baselines module
So that I can increase the confidence in the modules

Value / Purpose

As a follow-on from #6490 this ticket is to add missing unit tests for some of the sub-modules in the MP Baselines module.

There are roughly 9 resources for unit testing.

Useful Contacts

@richgreen-moj

Additional Information

Follow-on from #6490

Proposal / Unknowns

No response

Definition of Done

  • Missing unit tests added
  • New unit tests pass
  • Another team member has reviewed
@ep-93
Copy link
Contributor

ep-93 commented Jul 18, 2024

cloudtrail ✅ - write a follow-up story to test (8 resources) #7307

support ✅ - write a follow-up story to test (1 resource) #7307

These are the resources we need to test.

@mikereiddigital mikereiddigital self-assigned this Jul 22, 2024
@mikereiddigital mikereiddigital moved this from To Do to In Progress in Modernisation Platform Jul 22, 2024
@mikereiddigital
Copy link
Contributor

I'm picking up this one as I've not done much with unit tests & go before. If anyone wants to pair with me on it then do shout. :)

@sukeshreddyg sukeshreddyg self-assigned this Jul 23, 2024
@mikereiddigital
Copy link
Contributor

mikereiddigital commented Jul 23, 2024

Branch containing the completed support module unit test - https://github.com/ministryofjustice/modernisation-platform-terraform-baselines/tree/issue/7307. This has been tested locally against testing-test.

@sukeshreddyg
Copy link
Contributor

Added unit tests for the Support and CloudTrail modules.

ministryofjustice/modernisation-platform-terraform-baselines#547

@sukeshreddyg sukeshreddyg moved this from In Progress to For Review in Modernisation Platform Jul 29, 2024
@markgov
Copy link
Contributor

markgov commented Jul 31, 2024

just checked and the unit test is failing
/home/runner/work/modernisation-platform-terraform-baselines/modernisation-platform-terraform-baselines/test/go.mod:5: unknown directive: toolchain

@mikereiddigital
Copy link
Contributor

@markgov
Copy link
Contributor

markgov commented Jul 31, 2024

All looks good

@markgov markgov closed this as completed Jul 31, 2024
@github-project-automation github-project-automation bot moved this from For Review to Done in Modernisation Platform Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

5 participants