Page MenuHomePhabricator

Change bin/storage destroy to be less scary when removing test data
ClosedPublic

Authored by amckinley on Jul 25 2018, 7:08 PM.
Tags
None
Referenced Files
F13168940: D19535.diff
Tue, May 7, 8:03 AM
Unknown Object (File)
Fri, May 3, 7:47 AM
Unknown Object (File)
Thu, Apr 25, 1:43 AM
Unknown Object (File)
Thu, Apr 11, 10:01 AM
Unknown Object (File)
Apr 7 2024, 11:08 AM
Unknown Object (File)
Apr 5 2024, 4:05 AM
Unknown Object (File)
Jan 5 2024, 8:03 PM
Unknown Object (File)
Dec 23 2023, 7:12 AM
Subscribers

Details

Summary

I've pulled up this code probably three different times to make sure that the big scary warning does, in fact, still get printed even when passing --unitest-fixtures to bin/storage destroy. Make the warning message less scary if only removing test data.

Test Plan

Ran with and without --unitest-fixtures and saw expected warnings. After agreeing to warnings, test data was deleted as expected. Did not test bin/storage destroy without --unittest-fixtures.

Diff Detail

Repository
rP Phabricator
Branch
rm-storage
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20506
Build 27849: Run Core Tests
Build 27848: arc lint + arc unit