Apply suggestions from code review

Co-authored-by: Éric Gaspar <46165813+ericgaspar@users.noreply.github.com>
This commit is contained in:
Alexandre Aubin 2020-11-08 15:41:53 +01:00 committed by GitHub
parent e15a959860
commit c4e823062f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -732,7 +732,7 @@ class Manifest(TestSuite):
if "nonfree" in license.replace("-", ""): if "nonfree" in license.replace("-", ""):
yield Warning("'non-free' apps cannot be integrated in Yunohost's app catalog.") yield Warning("'non-free' apps cannot be integrated in YunoHost's app catalog.")
code_license = '<code property="spdx:licenseId">' + license + '</code>' code_license = '<code property="spdx:licenseId">' + license + '</code>'
@ -754,30 +754,30 @@ class Manifest(TestSuite):
def app_catalog(self): def app_catalog(self):
if not self.catalog_infos: if not self.catalog_infos:
yield Warning("This app is not Yunohost's application catalog") yield Warning("This app is not in YunoHost's application catalog")
@test() @test()
def app_catalog_revision(self): def app_catalog_revision(self):
if self.catalog_infos and self.catalog_infos.get("revision", "HEAD") != "HEAD": if self.catalog_infos and self.catalog_infos.get("revision", "HEAD") != "HEAD":
yield Error("You should make sure that the revision used in Yunohost's apps catalog is HEAD ...") yield Error("You should make sure that the revision used in YunoHost's apps catalog is HEAD...")
@test() @test()
def app_catalog_state(self): def app_catalog_state(self):
if self.catalog_infos and self.catalog_infos.get("state", "working") != "working": if self.catalog_infos and self.catalog_infos.get("state", "working") != "working":
yield Warning("The application is not flagged as working in Yunohost's apps catalog") yield Warning("The application is not flagged as working in YunoHost's apps catalog")
@test() @test()
def app_catalog_maintained(self): def app_catalog_maintained(self):
if self.catalog_infos and self.catalog_infos.get("maintained", True) is not True: if self.catalog_infos and self.catalog_infos.get("maintained", True) is not True:
yield Warning("The application is flagged as not maintained in Yunohost's apps catalog") yield Warning("The application is flagged as not maintained in YunoHost's apps catalog")
@test() @test()
def app_catalog_category(self): def app_catalog_category(self):
if self.catalog_infos and not self.catalog_infos.get("category"): if self.catalog_infos and not self.catalog_infos.get("category"):
yield Warning("The application has no associated category in Yunohost's apps catalog") yield Warning("The application has no associated category in YunoHost's apps catalog")
@test() @test()
def app_in_github_org(self): def app_in_github_org(self):