Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions test/src/lib/LibFs.buildFileForContract.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -135,9 +135,9 @@ contract LibFsBuildFileForContractTest is Test {
}

/// First generation for a contract is the normal case: nothing is at the
/// path yet. Removing a file that is not there reverts, so the existence
/// check in front of the removal is load bearing and this is what proves
/// it.
/// path yet. Removing a file that is not there reverts, so dropping the
/// existence check in front of the removal would make this case revert.
/// This test is what proves it.
function testBuildFileForContractFreshPath() external {
string memory name = "LibFsBuildFresh";
cleanup(name);
Expand Down
5 changes: 3 additions & 2 deletions test/src/lib/LibFs.buildFileForTaggedContract.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,9 @@ contract LibFsBuildFileForTaggedContractTest is Test {

/// The first generation for a tag has no directory yet, which is the normal
/// case for a release: the snapshot directory is created by the run that
/// fills it. `vm.writeFile` does not create a missing parent, so the
/// directory creation is load bearing and this is what proves it.
/// fills it. `vm.writeFile` does not create a missing parent, so dropping
/// the directory creation would make this case fail to write. This test is
/// what proves it.
function testBuildFileForTaggedContractCreatesTheTagDir() external {
string memory tag = "0_1_1$taggedFreshDir";
cleanup(tag);
Expand Down
Loading