diff options
| author | Richard Maw <richard.maw@gmail.com> | 2014-09-10 10:22:30 +0000 |
|---|---|---|
| committer | Richard Maw <richard.maw@gmail.com> | 2014-09-19 12:43:25 +0000 |
| commit | 59e1a82ab96115b6630269de28aadecf61d9bba9 (patch) | |
| tree | 1bab94717a4ea2ab6522996d2613f758485e7f63 /morphlib/builder2_tests.py | |
| parent | 90b3106de52d716f0ec08c5928a921db04b2eac8 (diff) | |
| download | morph-59e1a82ab96115b6630269de28aadecf61d9bba9.tar.gz | |
Eliminate BuilderBase.new_artifact
It was an odd thing to have, when Artifact objects are part of your
input.
Its purpose appears to have been to allow the build step to produce an
artifact called $morphology_name-rootfs, but since the split rules
decide that the artifact is called that anyway, the new_artifact step is
redundant.
Diffstat (limited to 'morphlib/builder2_tests.py')
| -rw-r--r-- | morphlib/builder2_tests.py | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/morphlib/builder2_tests.py b/morphlib/builder2_tests.py index d0d56b17..075134a8 100644 --- a/morphlib/builder2_tests.py +++ b/morphlib/builder2_tests.py @@ -1,4 +1,4 @@ -# Copyright (C) 2012-2013 Codethink Limited +# Copyright (C) 2012-2014 Codethink Limited # # This program is free software; you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -162,10 +162,6 @@ class BuilderBaseTests(unittest.TestCase): self.max_jobs, False) - def test_returns_an_artifact(self): - artifact = self.builder.new_artifact('le-artifact') - self.assertEqual(type(artifact), morphlib.artifact.Artifact) - def test_runs_desired_command(self): self.builder.runcmd(['foo', 'bar']) self.assertEqual(self.commands_run, [['foo', 'bar']]) |