From e4e08aca7910d1a8d75791db61de39fa03673fde Mon Sep 17 00:00:00 2001
From: John Firebaugh <john.firebaugh@gmail.com>
Date: Fri, 26 May 2017 10:23:12 -0700
Subject: [PATCH] [tests] Add test for calling addAnnotationImage with an
 existing ID

---
 test/api/annotations.test.cpp                 |  20 ++++++++++++++++--
 .../annotations/readd_image/expected.png      | Bin 0 -> 1031 bytes
 2 files changed, 18 insertions(+), 2 deletions(-)
 create mode 100644 test/fixtures/annotations/readd_image/expected.png

diff --git a/test/api/annotations.test.cpp b/test/api/annotations.test.cpp
index 9e85a74c5..4c27c871a 100644
--- a/test/api/annotations.test.cpp
+++ b/test/api/annotations.test.cpp
@@ -16,9 +16,12 @@ using namespace mbgl;
 
 namespace {
 
+PremultipliedImage namedImage(const std::string& name) {
+    return decodeImage(util::read_file("test/fixtures/sprites/" + name + ".png"));
+}
+
 std::unique_ptr<style::Image> namedMarker(const std::string& name) {
-    PremultipliedImage image = decodeImage(util::read_file("test/fixtures/sprites/" + name + ".png"));
-    return std::make_unique<style::Image>(name, std::move(image), 1.0);
+    return std::make_unique<style::Image>(name, namedImage(name), 1.0);
 }
 
 class AnnotationTest {
@@ -319,6 +322,19 @@ TEST(Annotations, SwitchStyle) {
     test.checkRendering("switch_style");
 }
 
+TEST(Annotations, ReaddImage) {
+    AnnotationTest test;
+
+    test.map.setStyleJSON(util::read_file("test/fixtures/api/empty.json"));
+    test.map.addAnnotationImage(namedMarker("default_marker"));
+    test.map.addAnnotation(SymbolAnnotation { Point<double> { 0, 0 }, "default_marker" });
+
+    test::render(test.map, test.view);
+
+    test.map.addAnnotationImage(std::make_unique<style::Image>("default_marker", namedImage("flipped_marker"), 1.0));
+    test.checkRendering("readd_image");
+}
+
 TEST(Annotations, QueryRenderedFeatures) {
     AnnotationTest test;
 
diff --git a/test/fixtures/annotations/readd_image/expected.png b/test/fixtures/annotations/readd_image/expected.png
new file mode 100644
index 0000000000000000000000000000000000000000..3c4847f3a70408ee864435e2a4f5c72181f0c6bc
GIT binary patch
literal 1031
zcmeAS@N?(olHy`uVBq!ia0y~yU<5K5893O0R7}x|GzJFdbDl1aAr*7p-r3(Rk}kuZ
zpg7@BR(|>Bqhdw(oE+8#Dsgx|RJ_5_v^KWu597+UK})?R2)xf>@l-Byy|jEI*AmW5
zrb#AIk(%sbFU>w%mZjW0^Tx88J-_U;&4<+Xncr(G??0OR{rH(TJBxu<4ixwxzj62O
z-w#uaB;USyQxo&I?vlvn?8QrTW8)v>)NF{&eElVNqRq+ZU7rqg9ee!o=PLR5cej2%
z&tsZ?mMy_oclNve`Ad2(y=4e12zvf6&7fkMg=Xx_G6wC4Yb*E5@y)l{TaY;G#71d`
zoEh6x5A418Vzs+`@3Qsz_iXxPp3hNb$eEDyKsS%6&QR`t?S=RIYb*c$`tttv_WsA~
z{q1Kzi&+PhOH<xhV%>0cZ#B=3?R)orznk#k`TFp6dtU!3eG;=ywfA_#hD0OjA7vIb
zUmhRr{{QMvX^Oo<rp>Qr-p_vvc7(70eLIigYn{WTY3uY0+}?e!+WWWe|Iz&L_1nt+
zSxr*QoV>hbZ=-R+h4=U4&A!I>Eb*Op=_{Mx%f$8DV|!nCFaPK3-T1`w-lebXeh(9}
zUq@bE_ums79RCh<z3Sb+eS2Q%K1P*(UdM>!-_g<bYmI)LKhG581`Gk&tJ(jzU9^85
ze`iOb%;`%vxEoBrEV%#w({u0XQr6$}udnl!_fxxk_?5-P$8)m1W*Y^Cz5K=?o5HOB
z{)c(#zmi|(=jZ!pTiVTujrN&ul*4vt*2-n?+`Ie#87)1ZVSoSU^L(bA+jhvzuobe=
zx+}M!dCgsM_s?s-8(-RPIcI6)ndzInwzI!Dz{~LO_$t5OzyFy3U#-U&<aP?IcGYFO
zyk9?-zgBzj=JA5#OK&rm6c|X{`I%^A_ci&zl!;)i8uP#YyLf5i{pYR=p7=^Gz0J5r
zLHWC9J<}zzuk)KF{wDT5-Tm*+)$r@*8z&#HXF9-kLM_hi^D=Yh+4uf<M@iRja5S2F
zzRFIy`r5bk&*pu9Zt=WGH`C{LqVrpRw)xM6@9()cTf8Q%u%u?)uCQHobV&Of*8OAT
W4;Hccaz<YtWSyt0pUXO@geCwPEu(e-

literal 0
HcmV?d00001