Add nil plugin #85

Merged
merged 4 commits into from Nov 13, 2015

Conversation

Projects
None yet
2 participants
Contributor

jocave commented Nov 6, 2015

The nil plugin is for parts in which all properties are optional.

snapcraft/plugins/nil.py
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+"""The nil plugin is useful for parts with entirely optional properties.
@sergiusens

sergiusens Nov 6, 2015

Collaborator

s/options/built in/

@@ -91,6 +91,7 @@ def test_list_plugins(self, mock_stdout):
jdk
make
maven
+nil
@sergiusens

sergiusens Nov 6, 2015

Collaborator

there is an integration test with a similar test you would need to update

+name: test-package
+version: 0.1
+vendor: Jonathan Cave <jonathan.cave@canonical.com>
+summary: one line summary
@sergiusens

sergiusens Nov 6, 2015

Collaborator

use summary to describe the test

+version: 0.1
+vendor: Jonathan Cave <jonathan.cave@canonical.com>
+summary: one line summary
+description: a longer description
@sergiusens

sergiusens Nov 6, 2015

Collaborator

use description for a longer description of the objective of the test

Collaborator

sergiusens commented Nov 6, 2015

Thanks for this, I made some in source comments

Additionally, can you remove the icon and just do touch icon.png, we don't do any checks for the icon and furthermore, the store won't be requiring an icon soon.

Additionally, I'd like @elopio to comment on the integration test, I think a test to see that it is indeed nil and does nothing is valuable (empty properties).

Collaborator

sergiusens commented Nov 6, 2015

Conflict prevent merging this, care to check?
Thanks.

integration-tests/units/jobs.pxu
@@ -401,17 +400,3 @@ command:
${SNAPCRAFT} shell xml2-config --cflags | grep -q `pwd`
${SNAPCRAFT} shell xslt-config --cflags | grep -q `pwd`
flags: simple has-leftovers
-
@sergiusens

sergiusens Nov 10, 2015

Collaborator

why were these removed?

jocave added some commits Nov 6, 2015

Add nil plugin
The nil plugin is for parts in which all properties are optional.
Improve nil plugin intergration tests
Add test for most simple use of nil plugin (an empty snap)
Update intregation-tests list plugin test

sergiusens added a commit that referenced this pull request Nov 13, 2015

@sergiusens sergiusens merged commit 8e62b40 into snapcore:master Nov 13, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.05%) to 91.282%
Details

@jocave jocave deleted the jocave:plugin-nil branch Mar 2, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment