[gentoo-commits] proj/pkgcore/pkgcore:master commit in: src/pkgcore/pytest/

[gentoo-commits] proj/pkgcore/pkgcore:master commit in: src/pkgcore/pytest/

[Arthur Zamarin]

commit:     2d586a926f128ec43d401b248f1b5a2db661abdf
Author:     Arthur Zamarin <arthurzam <AT> gentoo <DOT> org>
AuthorDate: Mon Nov  7 18:57:46 2022 +0000
Commit:     Arthur Zamarin <arthurzam <AT> gentoo <DOT> org>
CommitDate: Mon Nov  7 18:57:46 2022 +0000
URL:        https://gitweb.gentoo.org/proj/pkgcore/pkgcore.git/commit/?id=2d586a92

pytest.plugin: create_ebuild better homepage

Previous homepage was matching the MissingRemoteId check, so now use the
one from the project's github page, which doesn't flag it as missing.

Signed-off-by: Arthur Zamarin <arthurzam <AT> gentoo.org>

 src/pkgcore/pytest/plugin.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pkgcore/pytest/plugin.py b/src/pkgcore/pytest/plugin.py
index ddc12db6d..10739079d 100644
--- a/src/pkgcore/pytest/plugin.py
+++ b/src/pkgcore/pytest/plugin.py
@@ -226,7 +226,7 @@ class EbuildRepo:
         eapi = kwargs.pop('eapi', '7')
         slot = kwargs.pop('slot', '0')
         desc = kwargs.pop('description', 'stub package description')
-        homepage = kwargs.pop('homepage', 'https://github.com/pkgcore/pkgcheck')
+        homepage = kwargs.pop('homepage', 'https://pkgcore.github.io/pkgcheck')
         license = kwargs.pop('license', 'blank')
 
         ebuild_path = pjoin(ebuild_dir, f'{cpv.package}-{cpv.fullver}.ebuild')

[gentoo-commits] proj/pkgcore/pkgcore:master commit in: src/pkgcore/pytest/

[Arthur Zamarin]

commit:     aa1ec62ac69d2f89a9c3a5fdc5b40d5cff2c2c99
Author:     Arthur Zamarin <arthurzam <AT> gentoo <DOT> org>
AuthorDate: Thu Apr 20 18:39:35 2023 +0000
Commit:     Arthur Zamarin <arthurzam <AT> gentoo <DOT> org>
CommitDate: Thu Apr 20 18:41:12 2023 +0000
URL:        https://gitweb.gentoo.org/proj/pkgcore/pkgcore.git/commit/?id=aa1ec62a

pytest.plugin: blank line after EAPI declare in create_ebuild

Signed-off-by: Arthur Zamarin <arthurzam <AT> gentoo.org>

 src/pkgcore/pytest/plugin.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pkgcore/pytest/plugin.py b/src/pkgcore/pytest/plugin.py
index 142292bf4..6578050e2 100644
--- a/src/pkgcore/pytest/plugin.py
+++ b/src/pkgcore/pytest/plugin.py
@@ -260,7 +260,7 @@ class EbuildRepo:
                 """
                     )
                 )
-            f.write(f'EAPI="{eapi}"\n')
+            f.write(f"EAPI={eapi}\n\n")
             f.write(f'DESCRIPTION="{desc}"\n')
             f.write(f'HOMEPAGE="{homepage}"\n')
             f.write(f'SLOT="{slot}"\n')

[gentoo-commits] proj/pkgcore/pkgcore:master commit in: src/pkgcore/pytest/

[Arthur Zamarin]

commit:     1cf6d7caf82fc92aa72e4cd0d9c5815b160db5d3
Author:     Anthony Ryan <anthonyryan1 <AT> gmail <DOT> com>
AuthorDate: Sat Jan 13 04:27:03 2024 +0000
Commit:     Arthur Zamarin <arthurzam <AT> gentoo <DOT> org>
CommitDate: Mon Jan 15 06:32:50 2024 +0000
URL:        https://gitweb.gentoo.org/proj/pkgcore/pkgcore.git/commit/?id=1cf6d7ca

Write LICENSE before SLOT in create_ebuild()

Better matches the order defined in skel.ebuild

A prerequisite for https://github.com/pkgcore/pkgcheck/pull/645

Signed-off-by: Anthony Ryan <anthonyryan1 <AT> gmail.com>
Closes: https://github.com/pkgcore/pkgcore/pull/425
Signed-off-by: Arthur Zamarin <arthurzam <AT> gentoo.org>

 src/pkgcore/pytest/plugin.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/pkgcore/pytest/plugin.py b/src/pkgcore/pytest/plugin.py
index 6578050e2..0579a45ad 100644
--- a/src/pkgcore/pytest/plugin.py
+++ b/src/pkgcore/pytest/plugin.py
@@ -263,14 +263,14 @@ class EbuildRepo:
             f.write(f"EAPI={eapi}\n\n")
             f.write(f'DESCRIPTION="{desc}"\n')
             f.write(f'HOMEPAGE="{homepage}"\n')
-            f.write(f'SLOT="{slot}"\n')
-
             if license:
                 f.write(f'LICENSE="{license}"\n')
                 # create a fake license
                 os.makedirs(pjoin(self.path, "licenses"), exist_ok=True)
                 touch(pjoin(self.path, "licenses", license))
 
+            f.write(f'SLOT="{slot}"\n')
+
             for k, v in kwargs.items():
                 # handle sequences such as KEYWORDS and IUSE
                 if isinstance(v, (tuple, list)):

[gentoo-commits] proj/pkgcore/pkgcore:master commit in: src/pkgcore/pytest/

[Arthur Zamarin]

commit:     7a535fb3d2315fb6b73f3346fbd53468e48ccbfc
Author:     Brian Harring <ferringb <AT> gmail <DOT> com>
AuthorDate: Mon Jan 22 03:20:33 2024 +0000
Commit:     Arthur Zamarin <arthurzam <AT> gentoo <DOT> org>
CommitDate: Tue Jan 23 08:38:52 2024 +0000
URL:        https://gitweb.gentoo.org/proj/pkgcore/pkgcore.git/commit/?id=7a535fb3

fix: suppress $HOME for tests.

Suppressing $HOME is necessary for hygenic tests; to avoid
anything in the dev's environment becoming relied upon for
the test passing.

A slightly more salient point for me; every `git commit` test
was asking for my gpg unlock for git commits; obviously annoying,
also obviously unhygenic (I could have modified default git commit
template for example).

Signed-off-by: Brian Harring <ferringb <AT> gmail.com>

 src/pkgcore/pytest/plugin.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/pkgcore/pytest/plugin.py b/src/pkgcore/pytest/plugin.py
index 0579a45ad..d7231dd3c 100644
--- a/src/pkgcore/pytest/plugin.py
+++ b/src/pkgcore/pytest/plugin.py
@@ -41,6 +41,8 @@ class GitRepo:
                 self.add(pjoin(self.path, ".init"), create=True)
 
     def run(self, cmd, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, **kwargs):
+        env = os.environ.copy()
+        env["HOME"] = self.path
         return subprocess.run(
             cmd,
             cwd=self.path,
@@ -48,6 +50,7 @@ class GitRepo:
             check=True,
             stdout=stdout,
             stderr=stderr,
+            env=env,
             **kwargs,
         )