memory-analyzer unit test: include the header file, not cpp

There was a spurious include of the implementation file rather than the
header.

Author: tautschnig

unit/CMakeLists.txt

@@ -89,6 +89,9 @@ target_link_libraries(
         json-symtab-language
         statement-list
 )
+if(WITH_MEMORY_ANALYZER)
+  target_link_libraries(unit memory-analyzer-lib)
+endif()
 
 add_test(
     NAME unit

unit/Makefile

@@ -277,6 +277,8 @@ CATCH_TEST = unit_tests$(EXEEXT)
 EXCLUDED_TESTS=expr_undefined_casts.cpp
 ifneq ($(WITH_MEMORY_ANALYZER),1)
 EXCLUDED_TESTS += gdb_api.cpp
+else
+CPROVER_LIBS += ../src/memory-analyzer/gdb_api$(OBJEXT)
 endif
 ifeq ($(MINISAT2),)
 EXCLUDED_TESTS += satcheck_minisat2.cpp

unit/memory-analyzer/gdb_api.cpp

@@ -7,21 +7,19 @@ Author: Malte Mues <[email protected]>
 
 \*******************************************************************/
 
+#include <util/run.h>
+#include <util/tempfile.h>
+
+#include <memory-analyzer/gdb_api.h>
 #include <testing-utils/use_catch.h>
 
 #include <cstdio>
+#include <fstream>
+#include <iostream>
 #include <regex>
 #include <string>
 #include <vector>
 
-#include <fstream>
-#include <iostream>
-
-#include <memory-analyzer/gdb_api.cpp>
-
-#include <util/run.h>
-#include <util/tempfile.h>
-
 struct compile_test_filet
 {
   compile_test_filet() : compiled("test", "")