Skip to content

feat: export getStatus in startupOps. Write test for StartupOps.#49

Open
dallinjsevy wants to merge 3 commits into
trivirfrom
feature/push-restart
Open

feat: export getStatus in startupOps. Write test for StartupOps.#49
dallinjsevy wants to merge 3 commits into
trivirfrom
feature/push-restart

Conversation

@dallinjsevy
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@phalestrivir phalestrivir left a comment

Choose a reason for hiding this comment

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

Main change that needs to be made is that we'll need to refactor the ESV secrets tests like we did with variables so we can use the same test data for these tests. See comments below.

Comment thread src/ops/cloud/StartupOps.ts Outdated
Comment thread src/ops/cloud/StartupOps.test.ts Outdated
Comment thread src/ops/cloud/StartupOps.test.ts Outdated
Comment thread src/ops/cloud/StartupOps.test.ts Outdated
Comment thread src/ops/cloud/StartupOps.test.ts Outdated
Comment thread src/ops/cloud/StartupOps.test.ts Outdated
Comment thread src/test/snapshots/ops/cloud/StartupOps.test.js.snap
@@ -182,19 +182,17 @@ export async function stageVariable(variable: VariableSkeleton, create = true) {
// delete if exists, then create
try {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@dallinjsevy It looks like Volker made a similar change in commit 260cbd3. Can you resolve the conflict?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yes, I'll resolve the conflict today

@dallinjsevy dallinjsevy force-pushed the feature/push-restart branch from f9b7f42 to 418c185 Compare May 22, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants