Mercurial > hg > orthanc
annotate UnitTestsSources/ServerIndexTests.cpp @ 1286:b4acdb37e43b
refactoring
author | Sebastien Jodogne <s.jodogne@gmail.com> |
---|---|
date | Tue, 03 Feb 2015 16:51:19 +0100 |
parents | ee43f0d611a8 |
children | 63a6428771f4 |
rev | line source |
---|---|
827
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
1 /** |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
2 * Orthanc - A Lightweight, RESTful DICOM Store |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
3 * Copyright (C) 2012-2014 Medical Physics Department, CHU of Liege, |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
4 * Belgium |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
5 * |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
6 * This program is free software: you can redistribute it and/or |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
7 * modify it under the terms of the GNU General Public License as |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
8 * published by the Free Software Foundation, either version 3 of the |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
9 * License, or (at your option) any later version. |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
10 * |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
11 * In addition, as a special exception, the copyright holders of this |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
12 * program give permission to link the code of its release with the |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
13 * OpenSSL project's "OpenSSL" library (or with modified versions of it |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
14 * that use the same license as the "OpenSSL" library), and distribute |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
15 * the linked executables. You must obey the GNU General Public License |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
16 * in all respects for all of the code used other than "OpenSSL". If you |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
17 * modify file(s) with this exception, you may extend this exception to |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
18 * your version of the file(s), but you are not obligated to do so. If |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
19 * you do not wish to do so, delete this exception statement from your |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
20 * version. If you delete this exception statement from all source files |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
21 * in the program, then also delete it here. |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
22 * |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
23 * This program is distributed in the hope that it will be useful, but |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
24 * WITHOUT ANY WARRANTY; without even the implied warranty of |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
25 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
26 * General Public License for more details. |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
27 * |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
28 * You should have received a copy of the GNU General Public License |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
29 * along with this program. If not, see <http://www.gnu.org/licenses/>. |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
30 **/ |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
31 |
3d6f9b7d0add
precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
803
diff
changeset
|
32 |
831
84513f2ee1f3
pch for unit tests and server
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
827
diff
changeset
|
33 #include "PrecompiledHeadersUnitTests.h" |
181 | 34 #include "gtest/gtest.h" |
35 | |
183 | 36 #include "../OrthancServer/DatabaseWrapper.h" |
705 | 37 #include "../OrthancServer/ServerContext.h" |
38 #include "../OrthancServer/ServerIndex.h" | |
273
d384af918264
more detailed signal about deleted file
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
268
diff
changeset
|
39 #include "../Core/Uuid.h" |
711
8f62e8d5a384
test main dicom tags
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
707
diff
changeset
|
40 #include "../Core/DicomFormat/DicomNullValue.h" |
1124
790ff7a5b3bf
IStorageArea abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1030
diff
changeset
|
41 #include "../Core/FileStorage/FilesystemStorage.h" |
183 | 42 |
181 | 43 #include <ctype.h> |
44 #include <glog/logging.h> | |
521 | 45 #include <algorithm> |
181 | 46 |
183 | 47 using namespace Orthanc; |
181 | 48 |
183 | 49 namespace |
50 { | |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
51 enum DatabaseWrapperClass |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
52 { |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
53 DatabaseWrapperClass_SQLite |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
54 }; |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
55 |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
56 |
181 | 57 class ServerIndexListener : public IServerIndexListener |
58 { | |
59 public: | |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
60 std::vector<std::string> deletedFiles_; |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
61 std::vector<std::string> deletedResources_; |
183 | 62 std::string ancestorId_; |
63 ResourceType ancestorType_; | |
64 | |
65 void Reset() | |
181 | 66 { |
183 | 67 ancestorId_ = ""; |
68 deletedFiles_.clear(); | |
69 } | |
70 | |
71 virtual void SignalRemainingAncestor(ResourceType type, | |
72 const std::string& publicId) | |
73 { | |
74 ancestorId_ = publicId; | |
75 ancestorType_ = type; | |
181 | 76 } |
77 | |
273
d384af918264
more detailed signal about deleted file
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
268
diff
changeset
|
78 virtual void SignalFileDeleted(const FileInfo& info) |
181 | 79 { |
273
d384af918264
more detailed signal about deleted file
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
268
diff
changeset
|
80 const std::string fileUuid = info.GetUuid(); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
81 deletedFiles_.push_back(fileUuid); |
181 | 82 LOG(INFO) << "A file must be removed: " << fileUuid; |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
83 } |
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
84 |
1198 | 85 virtual void SignalChange(const ServerIndexChange& change) |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
86 { |
1198 | 87 if (change.GetChangeType() == ChangeType_Deleted) |
1177
5b2d8c280ac2
Plugins can monitor changes through callbacks
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1162
diff
changeset
|
88 { |
1198 | 89 deletedResources_.push_back(change.GetPublicId()); |
1177
5b2d8c280ac2
Plugins can monitor changes through callbacks
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1162
diff
changeset
|
90 } |
5b2d8c280ac2
Plugins can monitor changes through callbacks
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1162
diff
changeset
|
91 |
1198 | 92 LOG(INFO) << "Change related to resource " << change.GetPublicId() << " of type " |
93 << EnumerationToString(change.GetResourceType()) << ": " | |
94 << EnumerationToString(change.GetChangeType()); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
95 } |
1177
5b2d8c280ac2
Plugins can monitor changes through callbacks
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1162
diff
changeset
|
96 |
181 | 97 }; |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
98 |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
99 |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
100 class DatabaseWrapperTest : public ::testing::TestWithParam<DatabaseWrapperClass> |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
101 { |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
102 protected: |
738 | 103 std::auto_ptr<ServerIndexListener> listener_; |
1252 | 104 std::auto_ptr<IDatabaseWrapper> index_; |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
105 |
738 | 106 DatabaseWrapperTest() |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
107 { |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
108 } |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
109 |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
110 virtual void SetUp() |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
111 { |
738 | 112 listener_.reset(new ServerIndexListener); |
1247 | 113 index_.reset(new DatabaseWrapper()); |
114 index_->SetListener(*listener_); | |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
115 } |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
116 |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
117 virtual void TearDown() |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
118 { |
738 | 119 index_.reset(NULL); |
120 listener_.reset(NULL); | |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
121 } |
1286 | 122 |
123 void CheckTableRecordCount(uint32_t expected, const char* table) | |
124 { | |
125 if (GetParam() == DatabaseWrapperClass_SQLite) | |
126 { | |
127 DatabaseWrapper* sqlite = dynamic_cast<DatabaseWrapper*>(index_.get()); | |
128 ASSERT_EQ(expected, sqlite->GetTableRecordCount(table)); | |
129 } | |
130 } | |
131 | |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
132 }; |
181 | 133 } |
134 | |
135 | |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
136 INSTANTIATE_TEST_CASE_P(DatabaseWrapperName, |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
137 DatabaseWrapperTest, |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
138 ::testing::Values(DatabaseWrapperClass_SQLite)); |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
139 |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
140 |
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
141 TEST_P(DatabaseWrapperTest, Simple) |
181 | 142 { |
1286 | 143 DatabaseWrapper* sqlite_ = NULL; |
144 if (GetParam() == DatabaseWrapperClass_SQLite) | |
145 { | |
146 sqlite_ = dynamic_cast<DatabaseWrapper*>(index_.get()); | |
147 } | |
148 | |
181 | 149 int64_t a[] = { |
738 | 150 index_->CreateResource("a", ResourceType_Patient), // 0 |
151 index_->CreateResource("b", ResourceType_Study), // 1 | |
152 index_->CreateResource("c", ResourceType_Series), // 2 | |
153 index_->CreateResource("d", ResourceType_Instance), // 3 | |
154 index_->CreateResource("e", ResourceType_Instance), // 4 | |
155 index_->CreateResource("f", ResourceType_Instance), // 5 | |
156 index_->CreateResource("g", ResourceType_Study) // 6 | |
181 | 157 }; |
158 | |
738 | 159 ASSERT_EQ("a", index_->GetPublicId(a[0])); |
160 ASSERT_EQ("b", index_->GetPublicId(a[1])); | |
161 ASSERT_EQ("c", index_->GetPublicId(a[2])); | |
162 ASSERT_EQ("d", index_->GetPublicId(a[3])); | |
163 ASSERT_EQ("e", index_->GetPublicId(a[4])); | |
164 ASSERT_EQ("f", index_->GetPublicId(a[5])); | |
165 ASSERT_EQ("g", index_->GetPublicId(a[6])); | |
198
663cc6c46d0a
before refactoring of ServerIndex::GetXXX
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
197
diff
changeset
|
166 |
738 | 167 ASSERT_EQ(ResourceType_Patient, index_->GetResourceType(a[0])); |
168 ASSERT_EQ(ResourceType_Study, index_->GetResourceType(a[1])); | |
169 ASSERT_EQ(ResourceType_Series, index_->GetResourceType(a[2])); | |
170 ASSERT_EQ(ResourceType_Instance, index_->GetResourceType(a[3])); | |
171 ASSERT_EQ(ResourceType_Instance, index_->GetResourceType(a[4])); | |
172 ASSERT_EQ(ResourceType_Instance, index_->GetResourceType(a[5])); | |
173 ASSERT_EQ(ResourceType_Study, index_->GetResourceType(a[6])); | |
304 | 174 |
190 | 175 { |
1241 | 176 std::list<std::string> t; |
738 | 177 index_->GetAllPublicIds(t, ResourceType_Patient); |
190 | 178 |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
179 ASSERT_EQ(1u, t.size()); |
1241 | 180 ASSERT_EQ("a", t.front()); |
190 | 181 |
738 | 182 index_->GetAllPublicIds(t, ResourceType_Series); |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
183 ASSERT_EQ(1u, t.size()); |
1241 | 184 ASSERT_EQ("c", t.front()); |
190 | 185 |
738 | 186 index_->GetAllPublicIds(t, ResourceType_Study); |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
187 ASSERT_EQ(2u, t.size()); |
190 | 188 |
738 | 189 index_->GetAllPublicIds(t, ResourceType_Instance); |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
190 ASSERT_EQ(3u, t.size()); |
190 | 191 } |
192 | |
738 | 193 index_->SetGlobalProperty(GlobalProperty_FlushSleep, "World"); |
181 | 194 |
738 | 195 index_->AttachChild(a[0], a[1]); |
196 index_->AttachChild(a[1], a[2]); | |
197 index_->AttachChild(a[2], a[3]); | |
198 index_->AttachChild(a[2], a[4]); | |
199 index_->AttachChild(a[6], a[5]); | |
182 | 200 |
198
663cc6c46d0a
before refactoring of ServerIndex::GetXXX
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
197
diff
changeset
|
201 int64_t parent; |
738 | 202 ASSERT_FALSE(index_->LookupParent(parent, a[0])); |
203 ASSERT_TRUE(index_->LookupParent(parent, a[1])); ASSERT_EQ(a[0], parent); | |
204 ASSERT_TRUE(index_->LookupParent(parent, a[2])); ASSERT_EQ(a[1], parent); | |
205 ASSERT_TRUE(index_->LookupParent(parent, a[3])); ASSERT_EQ(a[2], parent); | |
206 ASSERT_TRUE(index_->LookupParent(parent, a[4])); ASSERT_EQ(a[2], parent); | |
207 ASSERT_TRUE(index_->LookupParent(parent, a[5])); ASSERT_EQ(a[6], parent); | |
208 ASSERT_FALSE(index_->LookupParent(parent, a[6])); | |
198
663cc6c46d0a
before refactoring of ServerIndex::GetXXX
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
197
diff
changeset
|
209 |
182 | 210 std::string s; |
1286 | 211 |
212 if (sqlite_ != NULL) | |
213 { | |
214 ASSERT_FALSE(sqlite_->GetParentPublicId(s, a[0])); | |
215 ASSERT_FALSE(sqlite_->GetParentPublicId(s, a[6])); | |
216 ASSERT_TRUE(sqlite_->GetParentPublicId(s, a[1])); ASSERT_EQ("a", s); | |
217 ASSERT_TRUE(sqlite_->GetParentPublicId(s, a[2])); ASSERT_EQ("b", s); | |
218 ASSERT_TRUE(sqlite_->GetParentPublicId(s, a[3])); ASSERT_EQ("c", s); | |
219 ASSERT_TRUE(sqlite_->GetParentPublicId(s, a[4])); ASSERT_EQ("c", s); | |
220 ASSERT_TRUE(sqlite_->GetParentPublicId(s, a[5])); ASSERT_EQ("g", s); | |
221 } | |
182 | 222 |
185 | 223 std::list<std::string> l; |
738 | 224 index_->GetChildrenPublicId(l, a[0]); ASSERT_EQ(1u, l.size()); ASSERT_EQ("b", l.front()); |
225 index_->GetChildrenPublicId(l, a[1]); ASSERT_EQ(1u, l.size()); ASSERT_EQ("c", l.front()); | |
226 index_->GetChildrenPublicId(l, a[3]); ASSERT_EQ(0u, l.size()); | |
227 index_->GetChildrenPublicId(l, a[4]); ASSERT_EQ(0u, l.size()); | |
228 index_->GetChildrenPublicId(l, a[5]); ASSERT_EQ(0u, l.size()); | |
229 index_->GetChildrenPublicId(l, a[6]); ASSERT_EQ(1u, l.size()); ASSERT_EQ("f", l.front()); | |
182 | 230 |
738 | 231 index_->GetChildrenPublicId(l, a[2]); ASSERT_EQ(2u, l.size()); |
182 | 232 if (l.front() == "d") |
233 { | |
234 ASSERT_EQ("e", l.back()); | |
235 } | |
236 else | |
237 { | |
238 ASSERT_EQ("d", l.back()); | |
239 ASSERT_EQ("e", l.front()); | |
240 } | |
241 | |
436
d51186bf7602
read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
310
diff
changeset
|
242 std::list<MetadataType> md; |
738 | 243 index_->ListAvailableMetadata(md, a[4]); |
436
d51186bf7602
read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
310
diff
changeset
|
244 ASSERT_EQ(0u, md.size()); |
d51186bf7602
read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
310
diff
changeset
|
245 |
738 | 246 index_->AddAttachment(a[4], FileInfo("my json file", FileContentType_DicomAsJson, 42, "md5", |
693
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
247 CompressionType_Zlib, 21, "compressedMD5")); |
738 | 248 index_->AddAttachment(a[4], FileInfo("my dicom file", FileContentType_Dicom, 42, "md5")); |
249 index_->AddAttachment(a[6], FileInfo("world", FileContentType_Dicom, 44, "md5")); | |
250 index_->SetMetadata(a[4], MetadataType_Instance_RemoteAet, "PINNACLE"); | |
436
d51186bf7602
read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
310
diff
changeset
|
251 |
738 | 252 index_->ListAvailableMetadata(md, a[4]); |
436
d51186bf7602
read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
310
diff
changeset
|
253 ASSERT_EQ(1u, md.size()); |
d51186bf7602
read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
310
diff
changeset
|
254 ASSERT_EQ(MetadataType_Instance_RemoteAet, md.front()); |
738 | 255 index_->SetMetadata(a[4], MetadataType_ModifiedFrom, "TUTU"); |
256 index_->ListAvailableMetadata(md, a[4]); | |
438 | 257 ASSERT_EQ(2u, md.size()); |
1030 | 258 |
259 std::map<MetadataType, std::string> md2; | |
260 index_->GetAllMetadata(md2, a[4]); | |
261 ASSERT_EQ(2u, md2.size()); | |
262 ASSERT_EQ("TUTU", md2[MetadataType_ModifiedFrom]); | |
263 ASSERT_EQ("PINNACLE", md2[MetadataType_Instance_RemoteAet]); | |
264 | |
738 | 265 index_->DeleteMetadata(a[4], MetadataType_ModifiedFrom); |
266 index_->ListAvailableMetadata(md, a[4]); | |
438 | 267 ASSERT_EQ(1u, md.size()); |
268 ASSERT_EQ(MetadataType_Instance_RemoteAet, md.front()); | |
182 | 269 |
1030 | 270 index_->GetAllMetadata(md2, a[4]); |
271 ASSERT_EQ(1u, md2.size()); | |
272 ASSERT_EQ("PINNACLE", md2[MetadataType_Instance_RemoteAet]); | |
273 | |
274 | |
738 | 275 ASSERT_EQ(21u + 42u + 44u, index_->GetTotalCompressedSize()); |
276 ASSERT_EQ(42u + 42u + 44u, index_->GetTotalUncompressedSize()); | |
181 | 277 |
1286 | 278 index_->SetMainDicomTag(a[3], DicomTag(0x0010, 0x0010), "PatientName"); |
181 | 279 |
280 int64_t b; | |
281 ResourceType t; | |
738 | 282 ASSERT_TRUE(index_->LookupResource("g", b, t)); |
181 | 283 ASSERT_EQ(7, b); |
284 ASSERT_EQ(ResourceType_Study, t); | |
285 | |
738 | 286 ASSERT_TRUE(index_->LookupMetadata(s, a[4], MetadataType_Instance_RemoteAet)); |
287 ASSERT_FALSE(index_->LookupMetadata(s, a[4], MetadataType_Instance_IndexInSeries)); | |
181 | 288 ASSERT_EQ("PINNACLE", s); |
1238 | 289 |
290 std::string u; | |
291 ASSERT_TRUE(index_->LookupMetadata(u, a[4], MetadataType_Instance_RemoteAet)); | |
292 ASSERT_EQ("PINNACLE", u); | |
293 ASSERT_FALSE(index_->LookupMetadata(u, a[4], MetadataType_Instance_IndexInSeries)); | |
181 | 294 |
738 | 295 ASSERT_TRUE(index_->LookupGlobalProperty(s, GlobalProperty_FlushSleep)); |
296 ASSERT_FALSE(index_->LookupGlobalProperty(s, static_cast<GlobalProperty>(42))); | |
181 | 297 ASSERT_EQ("World", s); |
298 | |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
299 FileInfo att; |
738 | 300 ASSERT_TRUE(index_->LookupAttachment(att, a[4], FileContentType_DicomAsJson)); |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
301 ASSERT_EQ("my json file", att.GetUuid()); |
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
302 ASSERT_EQ(21u, att.GetCompressedSize()); |
693
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
303 ASSERT_EQ("md5", att.GetUncompressedMD5()); |
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
304 ASSERT_EQ("compressedMD5", att.GetCompressedMD5()); |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
305 ASSERT_EQ(42u, att.GetUncompressedSize()); |
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
306 ASSERT_EQ(CompressionType_Zlib, att.GetCompressionType()); |
181 | 307 |
738 | 308 ASSERT_TRUE(index_->LookupAttachment(att, a[6], FileContentType_Dicom)); |
693
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
309 ASSERT_EQ("world", att.GetUuid()); |
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
310 ASSERT_EQ(44u, att.GetCompressedSize()); |
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
311 ASSERT_EQ("md5", att.GetUncompressedMD5()); |
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
312 ASSERT_EQ("md5", att.GetCompressedMD5()); |
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
313 ASSERT_EQ(44u, att.GetUncompressedSize()); |
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
314 ASSERT_EQ(CompressionType_None, att.GetCompressionType()); |
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
315 |
738 | 316 ASSERT_EQ(0u, listener_->deletedFiles_.size()); |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
317 ASSERT_EQ(0u, listener_->deletedResources_.size()); |
1286 | 318 |
319 CheckTableRecordCount(7, "Resources"); | |
320 CheckTableRecordCount(3, "AttachedFiles"); | |
321 CheckTableRecordCount(1, "Metadata"); | |
322 CheckTableRecordCount(1, "MainDicomTags"); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
323 |
738 | 324 index_->DeleteResource(a[0]); |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
325 ASSERT_EQ(5u, listener_->deletedResources_.size()); |
738 | 326 ASSERT_EQ(2u, listener_->deletedFiles_.size()); |
327 ASSERT_FALSE(std::find(listener_->deletedFiles_.begin(), | |
328 listener_->deletedFiles_.end(), | |
329 "my json file") == listener_->deletedFiles_.end()); | |
330 ASSERT_FALSE(std::find(listener_->deletedFiles_.begin(), | |
331 listener_->deletedFiles_.end(), | |
332 "my dicom file") == listener_->deletedFiles_.end()); | |
183 | 333 |
1286 | 334 CheckTableRecordCount(2, "Resources"); |
335 CheckTableRecordCount(0, "Metadata"); | |
336 CheckTableRecordCount(1, "AttachedFiles"); | |
337 CheckTableRecordCount(0, "MainDicomTags"); | |
338 | |
738 | 339 index_->DeleteResource(a[5]); |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
340 ASSERT_EQ(7u, listener_->deletedResources_.size()); |
1286 | 341 |
342 CheckTableRecordCount(0, "Resources"); | |
343 CheckTableRecordCount(0, "AttachedFiles"); | |
344 CheckTableRecordCount(2, "GlobalProperties"); | |
183 | 345 |
738 | 346 ASSERT_EQ(3u, listener_->deletedFiles_.size()); |
347 ASSERT_FALSE(std::find(listener_->deletedFiles_.begin(), | |
348 listener_->deletedFiles_.end(), | |
349 "world") == listener_->deletedFiles_.end()); | |
183 | 350 } |
351 | |
352 | |
353 | |
354 | |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
355 TEST_P(DatabaseWrapperTest, Upward) |
183 | 356 { |
1286 | 357 DatabaseWrapper* sqlite_ = NULL; |
358 if (GetParam() == DatabaseWrapperClass_SQLite) | |
359 { | |
360 sqlite_ = dynamic_cast<DatabaseWrapper*>(index_.get()); | |
361 } | |
362 | |
183 | 363 int64_t a[] = { |
738 | 364 index_->CreateResource("a", ResourceType_Patient), // 0 |
365 index_->CreateResource("b", ResourceType_Study), // 1 | |
366 index_->CreateResource("c", ResourceType_Series), // 2 | |
367 index_->CreateResource("d", ResourceType_Instance), // 3 | |
368 index_->CreateResource("e", ResourceType_Instance), // 4 | |
369 index_->CreateResource("f", ResourceType_Study), // 5 | |
370 index_->CreateResource("g", ResourceType_Series), // 6 | |
371 index_->CreateResource("h", ResourceType_Series) // 7 | |
183 | 372 }; |
373 | |
738 | 374 index_->AttachChild(a[0], a[1]); |
375 index_->AttachChild(a[1], a[2]); | |
376 index_->AttachChild(a[2], a[3]); | |
377 index_->AttachChild(a[2], a[4]); | |
378 index_->AttachChild(a[1], a[6]); | |
379 index_->AttachChild(a[0], a[5]); | |
380 index_->AttachChild(a[5], a[7]); | |
183 | 381 |
1286 | 382 if (sqlite_ != NULL) |
193
a1b9d1e1497b
failed attempt to compile with linux standard base
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
190
diff
changeset
|
383 { |
1241 | 384 std::list<std::string> j; |
1286 | 385 sqlite_->GetChildren(j, a[0]); |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
386 ASSERT_EQ(2u, j.size()); |
1241 | 387 ASSERT_TRUE((j.front() == "b" && j.back() == "f") || |
388 (j.back() == "b" && j.front() == "f")); | |
193
a1b9d1e1497b
failed attempt to compile with linux standard base
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
190
diff
changeset
|
389 |
1286 | 390 sqlite_->GetChildren(j, a[1]); |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
391 ASSERT_EQ(2u, j.size()); |
1241 | 392 ASSERT_TRUE((j.front() == "c" && j.back() == "g") || |
393 (j.back() == "c" && j.front() == "g")); | |
193
a1b9d1e1497b
failed attempt to compile with linux standard base
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
190
diff
changeset
|
394 |
1286 | 395 sqlite_->GetChildren(j, a[2]); |
232
5368bbe813cf
refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
218
diff
changeset
|
396 ASSERT_EQ(2u, j.size()); |
1241 | 397 ASSERT_TRUE((j.front() == "d" && j.back() == "e") || |
398 (j.back() == "d" && j.front() == "e")); | |
193
a1b9d1e1497b
failed attempt to compile with linux standard base
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
190
diff
changeset
|
399 |
1286 | 400 sqlite_->GetChildren(j, a[3]); ASSERT_EQ(0u, j.size()); |
401 sqlite_->GetChildren(j, a[4]); ASSERT_EQ(0u, j.size()); | |
402 sqlite_->GetChildren(j, a[5]); ASSERT_EQ(1u, j.size()); ASSERT_EQ("h", j.front()); | |
403 sqlite_->GetChildren(j, a[6]); ASSERT_EQ(0u, j.size()); | |
404 sqlite_->GetChildren(j, a[7]); ASSERT_EQ(0u, j.size()); | |
193
a1b9d1e1497b
failed attempt to compile with linux standard base
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
190
diff
changeset
|
405 } |
a1b9d1e1497b
failed attempt to compile with linux standard base
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
190
diff
changeset
|
406 |
738 | 407 listener_->Reset(); |
408 index_->DeleteResource(a[3]); | |
409 ASSERT_EQ("c", listener_->ancestorId_); | |
410 ASSERT_EQ(ResourceType_Series, listener_->ancestorType_); | |
183 | 411 |
738 | 412 listener_->Reset(); |
413 index_->DeleteResource(a[4]); | |
414 ASSERT_EQ("b", listener_->ancestorId_); | |
415 ASSERT_EQ(ResourceType_Study, listener_->ancestorType_); | |
183 | 416 |
738 | 417 listener_->Reset(); |
418 index_->DeleteResource(a[7]); | |
419 ASSERT_EQ("a", listener_->ancestorId_); | |
420 ASSERT_EQ(ResourceType_Patient, listener_->ancestorType_); | |
183 | 421 |
738 | 422 listener_->Reset(); |
423 index_->DeleteResource(a[6]); | |
424 ASSERT_EQ("", listener_->ancestorId_); // No more ancestor | |
181 | 425 } |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
426 |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
427 |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
428 TEST_P(DatabaseWrapperTest, PatientRecycling) |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
429 { |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
430 std::vector<int64_t> patients; |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
431 for (int i = 0; i < 10; i++) |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
432 { |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
433 std::string p = "Patient " + boost::lexical_cast<std::string>(i); |
738 | 434 patients.push_back(index_->CreateResource(p, ResourceType_Patient)); |
435 index_->AddAttachment(patients[i], FileInfo(p, FileContentType_Dicom, i + 10, | |
693
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
436 "md5-" + boost::lexical_cast<std::string>(i))); |
738 | 437 ASSERT_FALSE(index_->IsProtectedPatient(patients[i])); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
438 } |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
439 |
1286 | 440 CheckTableRecordCount(10u, "Resources"); |
441 CheckTableRecordCount(10u, "PatientRecyclingOrder"); | |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
442 |
738 | 443 listener_->Reset(); |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
444 ASSERT_EQ(0u, listener_->deletedResources_.size()); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
445 |
738 | 446 index_->DeleteResource(patients[5]); |
447 index_->DeleteResource(patients[0]); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
448 ASSERT_EQ(2u, listener_->deletedResources_.size()); |
1286 | 449 |
450 CheckTableRecordCount(8u, "Resources"); | |
451 CheckTableRecordCount(8u, "PatientRecyclingOrder"); | |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
452 |
738 | 453 ASSERT_EQ(2u, listener_->deletedFiles_.size()); |
454 ASSERT_EQ("Patient 5", listener_->deletedFiles_[0]); | |
455 ASSERT_EQ("Patient 0", listener_->deletedFiles_[1]); | |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
456 |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
457 int64_t p; |
738 | 458 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[1]); |
459 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
460 ASSERT_EQ(3u, listener_->deletedResources_.size()); |
738 | 461 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[2]); |
462 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
463 ASSERT_EQ(4u, listener_->deletedResources_.size()); |
738 | 464 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[3]); |
465 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
466 ASSERT_EQ(5u, listener_->deletedResources_.size()); |
738 | 467 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[4]); |
468 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
469 ASSERT_EQ(6u, listener_->deletedResources_.size()); |
738 | 470 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[6]); |
471 index_->DeleteResource(p); | |
472 index_->DeleteResource(patients[8]); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
473 ASSERT_EQ(8u, listener_->deletedResources_.size()); |
738 | 474 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[7]); |
475 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
476 ASSERT_EQ(9u, listener_->deletedResources_.size()); |
738 | 477 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[9]); |
478 index_->DeleteResource(p); | |
479 ASSERT_FALSE(index_->SelectPatientToRecycle(p)); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
480 ASSERT_EQ(10u, listener_->deletedResources_.size()); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
481 |
738 | 482 ASSERT_EQ(10u, listener_->deletedFiles_.size()); |
1286 | 483 |
484 CheckTableRecordCount(0, "Resources"); | |
485 CheckTableRecordCount(0, "PatientRecyclingOrder"); | |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
486 } |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
487 |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
488 |
737
1dee6e9bdbf4
abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
726
diff
changeset
|
489 TEST_P(DatabaseWrapperTest, PatientProtection) |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
490 { |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
491 std::vector<int64_t> patients; |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
492 for (int i = 0; i < 5; i++) |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
493 { |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
494 std::string p = "Patient " + boost::lexical_cast<std::string>(i); |
738 | 495 patients.push_back(index_->CreateResource(p, ResourceType_Patient)); |
496 index_->AddAttachment(patients[i], FileInfo(p, FileContentType_Dicom, i + 10, | |
693
01d8611c4a60
md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
632
diff
changeset
|
497 "md5-" + boost::lexical_cast<std::string>(i))); |
738 | 498 ASSERT_FALSE(index_->IsProtectedPatient(patients[i])); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
499 } |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
500 |
1286 | 501 CheckTableRecordCount(5, "Resources"); |
502 CheckTableRecordCount(5, "PatientRecyclingOrder"); | |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
503 |
738 | 504 ASSERT_FALSE(index_->IsProtectedPatient(patients[2])); |
505 index_->SetProtectedPatient(patients[2], true); | |
506 ASSERT_TRUE(index_->IsProtectedPatient(patients[2])); | |
1286 | 507 CheckTableRecordCount(5, "Resources"); |
508 CheckTableRecordCount(4, "PatientRecyclingOrder"); | |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
509 |
738 | 510 index_->SetProtectedPatient(patients[2], true); |
511 ASSERT_TRUE(index_->IsProtectedPatient(patients[2])); | |
1286 | 512 CheckTableRecordCount(4, "PatientRecyclingOrder"); |
738 | 513 index_->SetProtectedPatient(patients[2], false); |
514 ASSERT_FALSE(index_->IsProtectedPatient(patients[2])); | |
1286 | 515 CheckTableRecordCount(5, "PatientRecyclingOrder"); |
738 | 516 index_->SetProtectedPatient(patients[2], false); |
517 ASSERT_FALSE(index_->IsProtectedPatient(patients[2])); | |
1286 | 518 CheckTableRecordCount(5, "PatientRecyclingOrder"); |
519 CheckTableRecordCount(5, "Resources"); | |
738 | 520 index_->SetProtectedPatient(patients[2], true); |
521 ASSERT_TRUE(index_->IsProtectedPatient(patients[2])); | |
1286 | 522 CheckTableRecordCount(4, "PatientRecyclingOrder"); |
738 | 523 index_->SetProtectedPatient(patients[2], false); |
524 ASSERT_FALSE(index_->IsProtectedPatient(patients[2])); | |
1286 | 525 CheckTableRecordCount(5, "PatientRecyclingOrder"); |
738 | 526 index_->SetProtectedPatient(patients[3], true); |
527 ASSERT_TRUE(index_->IsProtectedPatient(patients[3])); | |
1286 | 528 CheckTableRecordCount(4, "PatientRecyclingOrder"); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
529 |
1286 | 530 CheckTableRecordCount(5, "Resources"); |
738 | 531 ASSERT_EQ(0u, listener_->deletedFiles_.size()); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
532 |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
533 // Unprotecting a patient puts it at the last position in the recycling queue |
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
534 int64_t p; |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
535 ASSERT_EQ(0u, listener_->deletedResources_.size()); |
738 | 536 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[0]); |
537 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
538 ASSERT_EQ(1u, listener_->deletedResources_.size()); |
738 | 539 ASSERT_TRUE(index_->SelectPatientToRecycle(p, patients[1])); ASSERT_EQ(p, patients[4]); |
540 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[1]); | |
541 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
542 ASSERT_EQ(2u, listener_->deletedResources_.size()); |
738 | 543 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[4]); |
544 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
545 ASSERT_EQ(3u, listener_->deletedResources_.size()); |
738 | 546 ASSERT_FALSE(index_->SelectPatientToRecycle(p, patients[2])); |
547 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[2]); | |
548 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
549 ASSERT_EQ(4u, listener_->deletedResources_.size()); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
550 // "patients[3]" is still protected |
738 | 551 ASSERT_FALSE(index_->SelectPatientToRecycle(p)); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
552 |
738 | 553 ASSERT_EQ(4u, listener_->deletedFiles_.size()); |
1286 | 554 CheckTableRecordCount(1, "Resources"); |
555 CheckTableRecordCount(0, "PatientRecyclingOrder"); | |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
556 |
738 | 557 index_->SetProtectedPatient(patients[3], false); |
1286 | 558 CheckTableRecordCount(1, "PatientRecyclingOrder"); |
738 | 559 ASSERT_FALSE(index_->SelectPatientToRecycle(p, patients[3])); |
560 ASSERT_TRUE(index_->SelectPatientToRecycle(p, patients[2])); | |
561 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[3]); | |
562 index_->DeleteResource(p); | |
1159
8f9d49192815
speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1134
diff
changeset
|
563 ASSERT_EQ(5u, listener_->deletedResources_.size()); |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
564 |
738 | 565 ASSERT_EQ(5u, listener_->deletedFiles_.size()); |
1286 | 566 CheckTableRecordCount(0, "Resources"); |
567 CheckTableRecordCount(0, "PatientRecyclingOrder"); | |
262
2354560daf2f
primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
252
diff
changeset
|
568 } |
310 | 569 |
570 | |
571 | |
1237 | 572 TEST(ServerIndex, Sequence) |
310 | 573 { |
1237 | 574 const std::string path = "UnitTestsStorage"; |
575 | |
576 Toolbox::RemoveFile(path + "/index"); | |
577 FilesystemStorage storage(path); | |
1247 | 578 DatabaseWrapper db; // The SQLite DB is in memory |
579 ServerContext context(db); | |
1237 | 580 context.SetStorageArea(storage); |
581 ServerIndex& index = context.GetIndex(); | |
582 | |
583 ASSERT_EQ(1u, index.IncrementGlobalSequence(GlobalProperty_AnonymizationSequence)); | |
584 ASSERT_EQ(2u, index.IncrementGlobalSequence(GlobalProperty_AnonymizationSequence)); | |
585 ASSERT_EQ(3u, index.IncrementGlobalSequence(GlobalProperty_AnonymizationSequence)); | |
586 ASSERT_EQ(4u, index.IncrementGlobalSequence(GlobalProperty_AnonymizationSequence)); | |
310 | 587 } |
521 | 588 |
589 | |
590 | |
1162 | 591 TEST_P(DatabaseWrapperTest, LookupIdentifier) |
521 | 592 { |
593 int64_t a[] = { | |
738 | 594 index_->CreateResource("a", ResourceType_Study), // 0 |
595 index_->CreateResource("b", ResourceType_Study), // 1 | |
596 index_->CreateResource("c", ResourceType_Study), // 2 | |
597 index_->CreateResource("d", ResourceType_Series) // 3 | |
521 | 598 }; |
599 | |
1286 | 600 index_->SetMainDicomTag(a[0], DICOM_TAG_STUDY_INSTANCE_UID, "0"); |
601 index_->SetMainDicomTag(a[1], DICOM_TAG_STUDY_INSTANCE_UID, "1"); | |
602 index_->SetMainDicomTag(a[2], DICOM_TAG_STUDY_INSTANCE_UID, "0"); | |
603 index_->SetMainDicomTag(a[3], DICOM_TAG_SERIES_INSTANCE_UID, "0"); | |
521 | 604 |
605 std::list<int64_t> s; | |
606 | |
1162 | 607 index_->LookupIdentifier(s, DICOM_TAG_STUDY_INSTANCE_UID, "0"); |
521 | 608 ASSERT_EQ(2u, s.size()); |
609 ASSERT_TRUE(std::find(s.begin(), s.end(), a[0]) != s.end()); | |
610 ASSERT_TRUE(std::find(s.begin(), s.end(), a[2]) != s.end()); | |
611 | |
1162 | 612 index_->LookupIdentifier(s, "0"); |
521 | 613 ASSERT_EQ(3u, s.size()); |
614 ASSERT_TRUE(std::find(s.begin(), s.end(), a[0]) != s.end()); | |
615 ASSERT_TRUE(std::find(s.begin(), s.end(), a[2]) != s.end()); | |
616 ASSERT_TRUE(std::find(s.begin(), s.end(), a[3]) != s.end()); | |
617 | |
1162 | 618 index_->LookupIdentifier(s, DICOM_TAG_STUDY_INSTANCE_UID, "1"); |
521 | 619 ASSERT_EQ(1u, s.size()); |
620 ASSERT_TRUE(std::find(s.begin(), s.end(), a[1]) != s.end()); | |
621 | |
1162 | 622 index_->LookupIdentifier(s, "1"); |
521 | 623 ASSERT_EQ(1u, s.size()); |
624 ASSERT_TRUE(std::find(s.begin(), s.end(), a[1]) != s.end()); | |
625 | |
626 | |
627 /*{ | |
628 std::list<std::string> s; | |
1162 | 629 context.GetIndex().LookupIdentifier(s, DICOM_TAG_STUDY_INSTANCE_UID, "1.2.250.1.74.20130819132500.29000036381059"); |
521 | 630 for (std::list<std::string>::iterator i = s.begin(); i != s.end(); i++) |
631 { | |
632 std::cout << "*** " << *i << std::endl;; | |
633 } | |
634 }*/ | |
562
f64e3838d6e1
refactoring enumerations
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
521
diff
changeset
|
635 } |
705 | 636 |
637 | |
638 | |
713
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
639 TEST(ServerIndex, AttachmentRecycling) |
705 | 640 { |
803
4689e400e0fa
directory to store the results of the unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
738
diff
changeset
|
641 const std::string path = "UnitTestsStorage"; |
1124
790ff7a5b3bf
IStorageArea abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1030
diff
changeset
|
642 |
705 | 643 Toolbox::RemoveFile(path + "/index"); |
1124
790ff7a5b3bf
IStorageArea abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1030
diff
changeset
|
644 FilesystemStorage storage(path); |
1247 | 645 DatabaseWrapper db; // The SQLite DB is in memory |
646 ServerContext context(db); | |
1134
ba9fd42284d0
preparing storage plugin
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
1124
diff
changeset
|
647 context.SetStorageArea(storage); |
705 | 648 ServerIndex& index = context.GetIndex(); |
707
203157cb4fde
unit tests of httpclient
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
706
diff
changeset
|
649 |
706 | 650 index.SetMaximumStorageSize(10); |
651 | |
705 | 652 Json::Value tmp; |
653 index.ComputeStatistics(tmp); | |
713
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
654 ASSERT_EQ(0, tmp["CountPatients"].asInt()); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
655 ASSERT_EQ(0, boost::lexical_cast<int>(tmp["TotalDiskSize"].asString())); |
705 | 656 |
657 ServerIndex::Attachments attachments; | |
658 | |
713
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
659 std::vector<std::string> ids; |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
660 for (int i = 0; i < 10; i++) |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
661 { |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
662 std::string id = boost::lexical_cast<std::string>(i); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
663 DicomMap instance; |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
664 instance.SetValue(DICOM_TAG_PATIENT_ID, "patient-" + id); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
665 instance.SetValue(DICOM_TAG_STUDY_INSTANCE_UID, "study-" + id); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
666 instance.SetValue(DICOM_TAG_SERIES_INSTANCE_UID, "series-" + id); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
667 instance.SetValue(DICOM_TAG_SOP_INSTANCE_UID, "instance-" + id); |
1006
649d47854314
proper handling of metadata in Store
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
831
diff
changeset
|
668 |
649d47854314
proper handling of metadata in Store
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
831
diff
changeset
|
669 std::map<MetadataType, std::string> instanceMetadata; |
649d47854314
proper handling of metadata in Store
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
831
diff
changeset
|
670 ServerIndex::MetadataMap metadata; |
649d47854314
proper handling of metadata in Store
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
831
diff
changeset
|
671 ASSERT_EQ(StoreStatus_Success, index.Store(instanceMetadata, instance, attachments, "", metadata)); |
649d47854314
proper handling of metadata in Store
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
831
diff
changeset
|
672 ASSERT_EQ(2, instanceMetadata.size()); |
1029 | 673 ASSERT_TRUE(instanceMetadata.find(MetadataType_Instance_RemoteAet) != instanceMetadata.end()); |
674 ASSERT_TRUE(instanceMetadata.find(MetadataType_Instance_ReceptionDate) != instanceMetadata.end()); | |
713
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
675 |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
676 DicomInstanceHasher hasher(instance); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
677 ids.push_back(hasher.HashPatient()); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
678 ids.push_back(hasher.HashStudy()); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
679 ids.push_back(hasher.HashSeries()); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
680 ids.push_back(hasher.HashInstance()); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
681 } |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
682 |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
683 index.ComputeStatistics(tmp); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
684 ASSERT_EQ(10, tmp["CountPatients"].asInt()); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
685 ASSERT_EQ(0, boost::lexical_cast<int>(tmp["TotalDiskSize"].asString())); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
686 |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
687 for (size_t i = 0; i < ids.size(); i++) |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
688 { |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
689 FileInfo info(Toolbox::GenerateUuid(), FileContentType_Dicom, 1, "md5"); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
690 index.AddAttachment(info, ids[i]); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
691 |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
692 index.ComputeStatistics(tmp); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
693 ASSERT_GE(10, boost::lexical_cast<int>(tmp["TotalDiskSize"].asString())); |
9d1973813d8b
test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
711
diff
changeset
|
694 } |
707
203157cb4fde
unit tests of httpclient
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
706
diff
changeset
|
695 |
203157cb4fde
unit tests of httpclient
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
706
diff
changeset
|
696 // Because the DB is in memory, the SQLite index must not have been created |
203157cb4fde
unit tests of httpclient
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
706
diff
changeset
|
697 ASSERT_THROW(Toolbox::GetFileSize(path + "/index"), OrthancException); |
705 | 698 } |