Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

repository files modified by "make check" #22925

Closed
qgib opened this issue Jun 7, 2016 · 4 comments
Closed

repository files modified by "make check" #22925

qgib opened this issue Jun 7, 2016 · 4 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Chore GitHub and other CI infrastructure changes

Comments

@qgib
Copy link
Contributor

qgib commented Jun 7, 2016

Author Name: Sandro Santilli (@strk)
Original Redmine Issue: 14976
Affected QGIS version: master
Redmine category:tests_suite
Assignee: Sandro Santilli


After running "make check", some testdata files are overridden:

1. Changes not staged for                                                 
1. modified:   python/plugins/processing/tests/testdata/custom/grass7/raster_4class.tif.aux.xml
1. modified:   python/plugins/processing/tests/testdata/custom/grass7/raster_5class.tif.aux.xml
1. modified:   python/plugins/processing/tests/testdata/custom/grass7/raster_6class.tif.aux.xml

They should either not be in the repository or not be overridden.
If the test implies modifying them, the tester should first copy them to temporary space and modify them there.

@qgib
Copy link
Contributor Author

qgib commented Jun 16, 2016

Author Name: Sandro Santilli (@strk)


Victor, is the testing framework for processing your responsibility ?


  • assigned_to_id was configured as Victor Olaya

@qgib
Copy link
Contributor Author

qgib commented Jun 16, 2016

Author Name: Sandro Santilli (@strk)


The diffs, for the record:

diff --git a/python/plugins/processing/tests/testdata/custom/grass7/raster_4class.tif.aux.xml b/python/plugins/processing/tests/testdata/custom/grass7/raster_4class.tif.aux.xml
index 129744d..9ada602 100644
--- a/python/plugins/processing/tests/testdata/custom/grass7/raster_4class.tif.aux.xml
+++ b/python/plugins/processing/tests/testdata/custom/grass7/raster_4class.tif.aux.xml
@@ -11,12 +11,12 @@
       </HistItem>
     </Histograms>
     <Metadata>
+      <MDI key="COLOR_TABLE_RULES_COUNT">5</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_0">1.000000e+00 1.600000e+00 255 255 0 0 255 0</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_1">1.600000e+00 2.200000e+00 0 255 0 0 255 255</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_2">2.200000e+00 2.800000e+00 0 255 255 0 0 255</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_3">2.800000e+00 3.400000e+00 0 0 255 255 0 255</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_4">3.400000e+00 4.000000e+00 255 0 255 255 0 0</MDI>
-      <MDI key="COLOR_TABLE_RULES_COUNT">5</MDI>
       <MDI key="Generated_with">GRASS GIS 7.0.3</MDI>
       <MDI key="STATISTICS_MAXIMUM">4</MDI>
       <MDI key="STATISTICS_MEAN">3.3133333333333</MDI>
diff --git a/python/plugins/processing/tests/testdata/custom/grass7/raster_5class.tif.aux.xml b/python/plugins/processing/tests/testdata/custom/grass7/raster_5class.tif.aux.xml
index ab04631..a5b37b9 100644
--- a/python/plugins/processing/tests/testdata/custom/grass7/raster_5class.tif.aux.xml
+++ b/python/plugins/processing/tests/testdata/custom/grass7/raster_5class.tif.aux.xml
@@ -11,12 +11,12 @@
       </HistItem>
     </Histograms>
     <Metadata>
+      <MDI key="COLOR_TABLE_RULES_COUNT">5</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_0">1.000000e+00 1.800000e+00 255 255 0 0 255 0</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_1">1.800000e+00 2.600000e+00 0 255 0 0 255 255</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_2">2.600000e+00 3.400000e+00 0 255 255 0 0 255</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_3">3.400000e+00 4.200000e+00 0 0 255 255 0 255</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_4">4.200000e+00 5.000000e+00 255 0 255 255 0 0</MDI>
-      <MDI key="COLOR_TABLE_RULES_COUNT">5</MDI>
       <MDI key="Generated_with">GRASS GIS 7.0.3</MDI>
       <MDI key="STATISTICS_MAXIMUM">5</MDI>
       <MDI key="STATISTICS_MEAN">3.4714760638298</MDI>
diff --git a/python/plugins/processing/tests/testdata/custom/grass7/raster_6class.tif.aux.xml b/python/plugins/processing/tests/testdata/custom/grass7/raster_6class.tif.aux.xml
index 4384222..40cd7dd 100644
--- a/python/plugins/processing/tests/testdata/custom/grass7/raster_6class.tif.aux.xml
+++ b/python/plugins/processing/tests/testdata/custom/grass7/raster_6class.tif.aux.xml
@@ -11,12 +11,12 @@
       </HistItem>
     </Histograms>
     <Metadata>
+      <MDI key="COLOR_TABLE_RULES_COUNT">5</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_0">1.000000e+00 2.000000e+00 255 255 0 0 255 0</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_1">2.000000e+00 3.000000e+00 0 255 0 0 255 255</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_2">3.000000e+00 4.000000e+00 0 255 255 0 0 255</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_3">4.000000e+00 5.000000e+00 0 0 255 255 0 255</MDI>
       <MDI key="COLOR_TABLE_RULE_RGB_4">5.000000e+00 6.000000e+00 255 0 255 255 0 0</MDI>
-      <MDI key="COLOR_TABLE_RULES_COUNT">5</MDI>
       <MDI key="Generated_with">GRASS GIS 7.0.3</MDI>
       <MDI key="STATISTICS_MAXIMUM">6</MDI>
       <MDI key="STATISTICS_MEAN">4.1343333333333</MDI>

I suspect those files should not be in the repository at all ?

@qgib
Copy link
Contributor Author

qgib commented Jun 16, 2016

Author Name: Sandro Santilli (@strk)


Fix ready in PR: #3206


  • assigned_to_id was changed from Victor Olaya to Sandro Santilli
  • status_id was changed from Open to In Progress

@qgib
Copy link
Contributor Author

qgib commented Jun 16, 2016

Author Name: Sandro Santilli (@strk)


Fixed in changeset "e241ab9b01e1ef2d4c57f2df0bd6b97ed7adc00f".


  • status_id was changed from In Progress to Closed

@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! Chore GitHub and other CI infrastructure changes labels May 25, 2019
@qgib qgib closed this as completed May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Chore GitHub and other CI infrastructure changes
Projects
None yet
Development

No branches or pull requests

1 participant