Skip to content

Commit

Permalink
Remove unnecessary setUp fixture from tarball package tests
Browse files Browse the repository at this point in the history
We initially included this fixture due to copy and pasting another
test. However, this `setUp` fixture isn't actually necessary for the
tarball dependency tests.
  • Loading branch information
QMalcolm committed Nov 14, 2023
1 parent 6d68e78 commit cb29a01
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions tests/functional/dependencies/test_simple_dependency.py
Original file line number Diff line number Diff line change
Expand Up @@ -379,10 +379,6 @@ def test_deps_bad_profile(self, project):


class TestSimpleDependcyTarball(object):
@pytest.fixture(scope="class", autouse=True)
def setUp(self, project):
project.run_sql_file(project.test_data_dir / Path("seed.sql"))

@pytest.fixture(scope="class")
def packages(self):
return {
Expand All @@ -400,10 +396,6 @@ def test_deps_simple_tarball_doesnt_error_out(self, project):


class TestBadTarballDependency(object):
@pytest.fixture(scope="class", autouse=True)
def setUp(self, project):
project.run_sql_file(project.test_data_dir / Path("seed.sql"))

def test_malformed_tarball_package_causes_exception(self, project):
# We have to specify the bad formatted package here because if we do it
# in a `packages` fixture, the test will blow up in the setup phase, meaning
Expand Down

0 comments on commit cb29a01

Please sign in to comment.