exif: Added test for ParseExifHeader.

- Tweaked the test names for correctness.
dsoprea · Jun 6, 2018 · 7fa077b · 7fa077b
1 parent 32664e6
commit 7fa077b
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 3 deletions.
diff --git a/assets/NDM_8901.jpg.exif b/assets/NDM_8901.jpg.exif
diff --git a/exif_test.go b/exif_test.go
@@ -8,6 +8,7 @@ import (
     "reflect"
 
     "io/ioutil"
+    "encoding/binary"
 
     "github.com/dsoprea/go-logging"
 )
@@ -29,7 +30,7 @@ func TestIsExif_False(t *testing.T) {
     }
 }
 
-func TestVisit(t *testing.T) {
+func TestExif_Visit(t *testing.T) {
     defer func() {
         if state := recover(); state != nil {
             err := log.Wrap(state.(error))
@@ -199,7 +200,7 @@ func TestVisit(t *testing.T) {
     }
 }
 
-func TestCollect(t *testing.T) {
+func TestExif_Collect(t *testing.T) {
     defer func() {
         if state := recover(); state != nil {
             err := log.Wrap(state.(error))
@@ -335,7 +336,30 @@ func TestCollect(t *testing.T) {
     }
 }
 
-func TestBuildAndParseExifHeader(t *testing.T) {
+func TestExif_ParseExifHeader(t *testing.T) {
+    filepath := path.Join(assetsPath, "NDM_8901.jpg.exif")
+
+    exifData, err := ioutil.ReadFile(filepath)
+    log.PanicIf(err)
+
+// TODO(dustin): !! We're currently built to expect the JPEG EXIF header-prefix, but our test-data doesn't have that.So, artificially prefix it, for now.
+    data := make([]byte, len(exifData) + len(ExifHeaderPrefixBytes))
+    copy(data[0:], ExifHeaderPrefixBytes)
+    copy(data[len(ExifHeaderPrefixBytes):], exifData)
+
+    e := NewExif()
+
+    eh, err := e.ParseExifHeader(data)
+    log.PanicIf(err)
+
+    if eh.ByteOrder != binary.LittleEndian {
+        t.Fatalf("Byte-order of EXIF header not correct.")
+    } else if eh.FirstIfdOffset != 0x8 {
+        t.Fatalf("First IFD offset not correct.")
+    }
+}
+
+func TestExif_BuildAndParseExifHeader(t *testing.T) {
     headerBytes, err := BuildExifHeader(TestDefaultByteOrder, 0x11223344)
     log.PanicIf(err)