Skip to content

Use Sus temporary directory fixture - #54

Merged
samuel-williams-shopify merged 1 commit into
mainfrom
use-sus-temporary-directory-context
Jun 21, 2026
Merged

Use Sus temporary directory fixture#54
samuel-williams-shopify merged 1 commit into
mainfrom
use-sus-temporary-directory-context

Conversation

@samuel-williams-shopify

Copy link
Copy Markdown
Contributor

Summary

  • Replace direct Dir.mktmpdir usage in command specs with Sus::Fixtures::TemporaryDirectoryContext.
  • Drop the now-unused tmpdir require.

Testing

  • git diff --check
  • bundle exec rubocop --format simple
  • GIT_CONFIG_COUNT=1 GIT_CONFIG_KEY_0=init.defaultBranch GIT_CONFIG_VALUE_0=main bundle exec sus test/utopia/command.rb

@samuel-williams-shopify
samuel-williams-shopify force-pushed the use-sus-temporary-directory-context branch 4 times, most recently from 679237a to 547c98d Compare June 21, 2026 11:37
Assisted-By: devx/648e2c48-7c87-49c5-b3db-7a3e62d0fbbc
@samuel-williams-shopify
samuel-williams-shopify force-pushed the use-sus-temporary-directory-context branch from 547c98d to cdfc767 Compare June 21, 2026 11:48
@samuel-williams-shopify
samuel-williams-shopify merged commit c742939 into main Jun 21, 2026
32 of 38 checks passed
@samuel-williams-shopify
samuel-williams-shopify deleted the use-sus-temporary-directory-context branch June 21, 2026 11:48
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.

1 participant