From 2fbe172dd125a3457b3974663c3adae63c35328e Mon Sep 17 00:00:00 2001
From: Tim Sutton <tim@linfiniti.com>
Date: Tue, 22 Jan 2013 11:07:50 +0200
Subject: [PATCH] Created test to replicate issue #7023

---
 tests/src/python/test_qgsrasterlayer.py       |  12 +++++
 .../raster/raster-pallette-crash2.qml         |  44 ++++++++++++++++++
 .../raster/raster-pallette-crash2.tif         | Bin 0 -> 2826 bytes
 .../raster/raster-pallette-crash2.tif.aux.xml |   5 ++
 4 files changed, 61 insertions(+)
 create mode 100644 tests/testdata/raster/raster-pallette-crash2.qml
 create mode 100644 tests/testdata/raster/raster-pallette-crash2.tif
 create mode 100644 tests/testdata/raster/raster-pallette-crash2.tif.aux.xml

diff --git a/tests/src/python/test_qgsrasterlayer.py b/tests/src/python/test_qgsrasterlayer.py
index 23682480cde..3f02e5df9a2 100644
--- a/tests/src/python/test_qgsrasterlayer.py
+++ b/tests/src/python/test_qgsrasterlayer.py
@@ -151,6 +151,18 @@ class TestQgsRasterLayer(TestCase):
         myResultFlag = myChecker.runTest("raster_transparency_python");
         assert myResultFlag, "Raster transparency rendering test failed"
 
+    def testIssue7023(self):
+        """Check if converting a raster from 1.8 to 2 works."""
+        myPath = os.path.join(unitTestDataPath('raster'),
+                              'raster-pallette-crash2.tif')
+        myFileInfo = QFileInfo(myPath)
+        myBaseName = myFileInfo.baseName()
+        myRasterLayer = QgsRasterLayer(myPath, myBaseName)
+        myMessage = 'Raster not loaded: %s' % myPath
+        assert myRasterLayer.isValid(), myMessage
+        # crash on next line
+        QgsMapLayerRegistry.addMapLayers([myRasterLayer])
+
     def testShaderCrash(self):
         """Check if we assign a shader and then reassign it no crash occurs."""
         myPath = os.path.join(unitTestDataPath('raster'),
diff --git a/tests/testdata/raster/raster-pallette-crash2.qml b/tests/testdata/raster/raster-pallette-crash2.qml
new file mode 100644
index 00000000000..2800ad494be
--- /dev/null
+++ b/tests/testdata/raster/raster-pallette-crash2.qml
@@ -0,0 +1,44 @@
+<!DOCTYPE qgis PUBLIC 'http://mrcc.com/qgis.dtd' 'SYSTEM'>
+<qgis version="1.8.0-Lisboa" minimumScale="0" maximumScale="1e+08" hasScaleBasedVisibilityFlag="0">
+  <transparencyLevelInt>255</transparencyLevelInt>
+  <rasterproperties>
+    <mDrawingStyle>PalettedColor</mDrawingStyle>
+    <mColorShadingAlgorithm>ColorRampShader</mColorShadingAlgorithm>
+    <mInvertColor boolean="false"/>
+    <mRedBandName>Not Set</mRedBandName>
+    <mGreenBandName>Not Set</mGreenBandName>
+    <mBlueBandName>Not Set</mBlueBandName>
+    <mGrayBandName>Band 1</mGrayBandName>
+    <mStandardDeviations>0</mStandardDeviations>
+    <mUserDefinedRGBMinimumMaximum boolean="false"/>
+    <mRGBMinimumMaximumEstimated boolean="true"/>
+    <mUserDefinedGrayMinimumMaximum boolean="false"/>
+    <mGrayMinimumMaximumEstimated boolean="true"/>
+    <mContrastEnhancementAlgorithm>NoEnhancement</mContrastEnhancementAlgorithm>
+    <contrastEnhancementMinMaxValues>
+      <minMaxEntry>
+        <min>-1.79769e+308</min>
+        <max>1.79769e+308</max>
+      </minMaxEntry>
+    </contrastEnhancementMinMaxValues>
+    <mNoDataValue mValidNoDataValue="true">nan</mNoDataValue>
+    <singleValuePixelList>
+      <pixelListEntry pixelValue="0.419576" percentTransparent="100"/>
+      <pixelListEntry pixelValue="94.209307" percentTransparent="30"/>
+      <pixelListEntry pixelValue="187.999039" percentTransparent="30"/>
+      <pixelListEntry pixelValue="281.788770" percentTransparent="30"/>
+      <pixelListEntry pixelValue="375.578502" percentTransparent="30"/>
+    </singleValuePixelList>
+    <threeValuePixelList>
+      <pixelListEntry red="nan" blue="nan" green="nan" percentTransparent="100"/>
+    </threeValuePixelList>
+    <customColorRamp>
+      <colorRampType>DISCRETE</colorRampType>
+      <colorRampEntry red="238" blue="238" value="0.419576" green="255" label="0.42 people/cell"/>
+      <colorRampEntry red="255" blue="127" value="94.209307" green="255" label=""/>
+      <colorRampEntry red="225" blue="0" value="187.999039" green="85" label="188.00 people/cell"/>
+      <colorRampEntry red="228" blue="27" value="281.788770" green="0" label=""/>
+      <colorRampEntry red="115" blue="0" value="375.578502" green="0" label="375.58 people/cell"/>
+    </customColorRamp>
+  </rasterproperties>
+</qgis>
diff --git a/tests/testdata/raster/raster-pallette-crash2.tif b/tests/testdata/raster/raster-pallette-crash2.tif
new file mode 100644
index 0000000000000000000000000000000000000000..de309a93fb2092e0a2df2d6b54a064a3e9a998e4
GIT binary patch
literal 2826
zcmcJMJxjxI5XLWQsVx?@iVD(VvEbk+3WA_<s-@skp^B647Aa1}#TST^=%AmVINEk_
z6ZCU9iG%JgzR<x-%md{h{{KmIXnyzd+;f+-RGOhA5oL&s7#Tki8D2chx#`6g$1yK<
zbcvogW5nIr1~*1~>SJE)_>6m+^Au6GM^o|Q7;6TO)8z1xe@7>2N-*E}*Ro@jXHKzy
z+fI|o@eaow+jOt5b9~q|DbDrUc8#Az-R}F-ZMnP*y2W1}jf>RB{N<K4JMnq<T$+4<
z)NtRdj~sJ@H}h*Sc_Du`gI<P=WXI+%CEWb1kV>{~r{C|A=yZ4gD%Y%AEU~WFH&&{}
zN`7JPfC`1;;@Wi9S{kX|Yz~?h@}LYlf8YAbt-kJAplTj{P~L|+DEq1$cR&}`2Xm6}
zm6UtY*PVynkJ1M@^w<xg4nnU>%z>&o?3KhksPaJ{dJuI#%Dt*w*H`LfF7Hw8m3@_;
z>?4=_t2?A`)Lg!=ZXR{7-@NXA=<r?r{G!&eM@8HLM%{~f_+dZlAnG9WlF<JbRr_`C
rfiCQwx_5;24?2hUkqqh+RxkI;JgjfnIk{iv@Wua#T$k|k)5G^0h6u(#

literal 0
HcmV?d00001

diff --git a/tests/testdata/raster/raster-pallette-crash2.tif.aux.xml b/tests/testdata/raster/raster-pallette-crash2.tif.aux.xml
new file mode 100644
index 00000000000..00063f7bf34
--- /dev/null
+++ b/tests/testdata/raster/raster-pallette-crash2.tif.aux.xml
@@ -0,0 +1,5 @@
+<PAMDataset>
+  <PAMRasterBand band="1">
+    <NoDataValue le_hex_equiv="000000000000F87F">nan</NoDataValue>
+  </PAMRasterBand>
+</PAMDataset>