511dd8aa75f9148415c8f9efc91257c62e503e2e,api/python/tests/integration/test_packages.py,PackageTest,test_set_package_entry_as_object,#PackageTest#,563

Before Change

        Package.delete_local_file(pkg.get("mydataframe5.csv"))
        Package.delete_local_file(pkg.get("mydataframe6.tsv"))

        for lk, entry in pkg.walk():
            file_path = parse_file_url(urlparse(entry.physical_keys[0]))
            assert not (pathlib.Path(file_path)).exists(), "The serialization files should have been deleted"

            self.file_sweeper_path_list.append(file_path)


    def test_tophash_changes(self):
        test_file = Path("test.txt")
        test_file.write_text("asdf", "utf-8")

After Change

                                                   "was serialized"

        // Test that push cleans up the temporary files, if and only if the serialization_location was not set
        with patch("botocore.client.BaseClient._make_api_call", new=mock_make_api_call), \
            patch("quilt3.Package._materialize") as materialize_mock, \
            patch("quilt3.Package.build") as build_mock:
            materialize_mock.return_value = pkg

            pkg.push("Quilt/test_pkg_name", "s3://test-bucket")

        for lk in ["mydataframe1.parquet", "mydataframe2.csv", "mydataframe3.tsv"]:
            file_path = parse_file_url(urlparse(pkg.get(lk)))
            assert pathlib.Path(file_path).exists(), "These files should not have been deleted during push()"

Instances