[gem5-dev] Change in gem5/gem5[develop]: util-gem5art: Fix the artifact test with incorrect git repo

2021-07-07 Thread Bobby R. Bruce (Gerrit) via gem5-dev
Bobby R. Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/47220 )


Change subject: util-gem5art: Fix the artifact test with incorrect git repo
..

util-gem5art: Fix the artifact test with incorrect git repo

The gem5art tests add the current repository as an artifact. Previously
this repository was called "gem5art". Since moving to the gem5 repo, it
is now called "gem5". Therefore the test checking the repository name
needed fixed.

Note: We need a better solution for testing gem5art's git functionality.
Using the current repository is not stable, as this fix shows. Noted
in Jira: https://gem5.atlassian.net/browse/GEM5-1013.

Change-Id: I017ff039ccb1c0a290f9aa1a09bd738c1820f88f
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/47220
Tested-by: kokoro 
Reviewed-by: Jason Lowe-Power 
Maintainer: Jason Lowe-Power 
---
M util/gem5art/artifact/tests/test_artifact.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/util/gem5art/artifact/tests/test_artifact.py  
b/util/gem5art/artifact/tests/test_artifact.py

index 6a54650..897113d 100644
--- a/util/gem5art/artifact/tests/test_artifact.py
+++ b/util/gem5art/artifact/tests/test_artifact.py
@@ -92,7 +92,7 @@
 def test_origin(self):
 git = artifact.artifact.getGit(Path("."))
 self.assertTrue(
-git["origin"].endswith("gem5art"), "Origin should end with  
gem5art"
+git["origin"].endswith("gem5"), "Origin should end with  
gem5art"

 )





2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the  
submitted one.

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/47220
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I017ff039ccb1c0a290f9aa1a09bd738c1820f88f
Gerrit-Change-Number: 47220
Gerrit-PatchSet: 6
Gerrit-Owner: Bobby R. Bruce 
Gerrit-Reviewer: Bobby R. Bruce 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: kokoro 
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: util-gem5art: Fix the artifact test with incorrect git repo

2021-06-25 Thread Bobby R. Bruce (Gerrit) via gem5-dev
Bobby R. Bruce has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/47220 )



Change subject: util-gem5art: Fix the artifact test with incorrect git repo
..

util-gem5art: Fix the artifact test with incorrect git repo

The gem5art tests add the current repository as an artifact. Previously
this repository was called "gem5art". Since moving to the gem5 repo, it
is now called "gem5". Therefore the test checking the repository name
needed fixed.

Note: We need a better solution for testing gem5art's git functionality.
Using the current repository is not stable, as this fix shows.

Change-Id: I017ff039ccb1c0a290f9aa1a09bd738c1820f88f
---
M util/gem5art/artifact/tests/test_artifact.py
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/util/gem5art/artifact/tests/test_artifact.py  
b/util/gem5art/artifact/tests/test_artifact.py

index 6a54650..897113d 100644
--- a/util/gem5art/artifact/tests/test_artifact.py
+++ b/util/gem5art/artifact/tests/test_artifact.py
@@ -92,7 +92,7 @@
 def test_origin(self):
 git = artifact.artifact.getGit(Path("."))
 self.assertTrue(
-git["origin"].endswith("gem5art"), "Origin should end with  
gem5art"
+git["origin"].endswith("gem5"), "Origin should end with  
gem5art"

 )



--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/47220
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I017ff039ccb1c0a290f9aa1a09bd738c1820f88f
Gerrit-Change-Number: 47220
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby R. Bruce 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s