annotate UnitTestsSources/ServerIndexTests.cpp @ 2826:c277e0421200

unit testing of overwriting
author Sebastien Jodogne <s.jodogne@gmail.com>
date Wed, 19 Sep 2018 16:06:41 +0200
parents 925d8dc03a23
children 859a4950d48f
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
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
1900
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1823
diff changeset
3 * Copyright (C) 2012-2016 Sebastien Jodogne, Medical Physics
1288
6e7e5ed91c2d upgrade to year 2015
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1287
diff changeset
4 * Department, University Hospital of Liege, Belgium
2447
878b59270859 upgrade to year 2018
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2244
diff changeset
5 * Copyright (C) 2017-2018 Osimis S.A., Belgium
827
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
6 *
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
7 * 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
8 * 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
9 * 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
10 * License, or (at your option) any later version.
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
11 *
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
12 * 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
13 * 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
14 * 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
15 * 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
16 * 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
17 * 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
18 * 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
19 * 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
20 * 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
21 * 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
22 * in the program, then also delete it here.
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
23 *
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
24 * 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
25 * 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
26 * 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
27 * General Public License for more details.
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
28 *
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
29 * 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
30 * 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
31 **/
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
32
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 803
diff changeset
33
831
84513f2ee1f3 pch for unit tests and server
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 827
diff changeset
34 #include "PrecompiledHeadersUnitTests.h"
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
35 #include "gtest/gtest.h"
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
36
1486
f967bdf8534e refactoring to Logging.h
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1453
diff changeset
37 #include "../Core/FileStorage/FilesystemStorage.h"
2826
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
38 #include "../Core/FileStorage/MemoryStorageArea.h"
1486
f967bdf8534e refactoring to Logging.h
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1453
diff changeset
39 #include "../Core/Logging.h"
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
40 #include "../OrthancServer/DatabaseWrapper.h"
705
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
41 #include "../OrthancServer/ServerContext.h"
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
42 #include "../OrthancServer/ServerIndex.h"
1747
ca69082ab200 reorganization
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1746
diff changeset
43 #include "../OrthancServer/Search/LookupIdentifierQuery.h"
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
44
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
45 #include <ctype.h>
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
46 #include <algorithm>
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
47
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
48 using namespace Orthanc;
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
49
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
50 namespace
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
51 {
737
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
52 enum DatabaseWrapperClass
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
53 {
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
54 DatabaseWrapperClass_SQLite
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
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
57
1432
0ac74fa21db8 rename IServerIndexListener as IDatabaseListener
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1294
diff changeset
58 class TestDatabaseListener : public IDatabaseListener
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
59 {
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
60 public:
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
61 std::vector<std::string> deletedFiles_;
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
62 std::vector<std::string> deletedResources_;
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
63 std::string ancestorId_;
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
64 ResourceType ancestorType_;
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
65
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
66 void Reset()
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
67 {
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
68 ancestorId_ = "";
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
69 deletedFiles_.clear();
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
70 }
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
71
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
72 virtual void SignalRemainingAncestor(ResourceType type,
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
73 const std::string& publicId)
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
74 {
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
75 ancestorId_ = publicId;
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
76 ancestorType_ = type;
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
77 }
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
78
273
d384af918264 more detailed signal about deleted file
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 268
diff changeset
79 virtual void SignalFileDeleted(const FileInfo& info)
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
80 {
273
d384af918264 more detailed signal about deleted file
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 268
diff changeset
81 const std::string fileUuid = info.GetUuid();
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
82 deletedFiles_.push_back(fileUuid);
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
83 LOG(INFO) << "A file must be removed: " << fileUuid;
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
84 }
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
85
1198
1169528a9a5f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1177
diff changeset
86 virtual void SignalChange(const ServerIndexChange& change)
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
87 {
1198
1169528a9a5f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1177
diff changeset
88 if (change.GetChangeType() == ChangeType_Deleted)
1177
5b2d8c280ac2 Plugins can monitor changes through callbacks
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1162
diff changeset
89 {
1198
1169528a9a5f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1177
diff changeset
90 deletedResources_.push_back(change.GetPublicId());
1177
5b2d8c280ac2 Plugins can monitor changes through callbacks
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1162
diff changeset
91 }
5b2d8c280ac2 Plugins can monitor changes through callbacks
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1162
diff changeset
92
1198
1169528a9a5f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1177
diff changeset
93 LOG(INFO) << "Change related to resource " << change.GetPublicId() << " of type "
1169528a9a5f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1177
diff changeset
94 << EnumerationToString(change.GetResourceType()) << ": "
1169528a9a5f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1177
diff changeset
95 << EnumerationToString(change.GetChangeType());
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
96 }
1177
5b2d8c280ac2 Plugins can monitor changes through callbacks
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1162
diff changeset
97
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
98 };
737
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
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
101 class DatabaseWrapperTest : public ::testing::TestWithParam<DatabaseWrapperClass>
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
102 {
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
103 protected:
1432
0ac74fa21db8 rename IServerIndexListener as IDatabaseListener
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1294
diff changeset
104 std::auto_ptr<TestDatabaseListener> listener_;
1252
ee43f0d611a8 testing
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1247
diff changeset
105 std::auto_ptr<IDatabaseWrapper> index_;
737
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
106
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
107 DatabaseWrapperTest()
737
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
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
111 virtual void SetUp()
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
112 {
1432
0ac74fa21db8 rename IServerIndexListener as IDatabaseListener
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1294
diff changeset
113 listener_.reset(new TestDatabaseListener);
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
114
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
115 switch (GetParam())
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
116 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
117 case DatabaseWrapperClass_SQLite:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
118 index_.reset(new DatabaseWrapper());
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
119 break;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
120
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
121 default:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
122 throw OrthancException(ErrorCode_InternalError);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
123 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
124
1247
32fcc5dc7562 abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1241
diff changeset
125 index_->SetListener(*listener_);
1672
4c5a85c3ff43 sample database plugin now working
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1549
diff changeset
126 index_->Open();
737
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
127 }
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
128
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
129 virtual void TearDown()
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
130 {
1672
4c5a85c3ff43 sample database plugin now working
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1549
diff changeset
131 index_->Close();
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
132 index_.reset(NULL);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
133 listener_.reset(NULL);
737
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
134 }
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
135
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
136 void CheckTableRecordCount(uint32_t expected, const char* table)
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
137 {
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
138 switch (GetParam())
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
139 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
140 case DatabaseWrapperClass_SQLite:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
141 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
142 DatabaseWrapper* sqlite = dynamic_cast<DatabaseWrapper*>(index_.get());
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
143 ASSERT_EQ(expected, sqlite->GetTableRecordCount(table));
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
144 break;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
145 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
146
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
147 default:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
148 throw OrthancException(ErrorCode_InternalError);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
149 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
150 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
151
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
152 void CheckNoParent(int64_t id)
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
153 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
154 std::string s;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
155
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
156 switch (GetParam())
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
157 {
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
158 case DatabaseWrapperClass_SQLite:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
159 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
160 DatabaseWrapper* sqlite = dynamic_cast<DatabaseWrapper*>(index_.get());
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
161 ASSERT_FALSE(sqlite->GetParentPublicId(s, id));
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
162 break;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
163 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
164
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
165 default:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
166 throw OrthancException(ErrorCode_InternalError);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
167 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
168 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
169
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
170 void CheckParentPublicId(const char* expected, int64_t id)
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
171 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
172 std::string s;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
173
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
174 switch (GetParam())
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
175 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
176 case DatabaseWrapperClass_SQLite:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
177 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
178 DatabaseWrapper* sqlite = dynamic_cast<DatabaseWrapper*>(index_.get());
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
179 ASSERT_TRUE(sqlite->GetParentPublicId(s, id));
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
180 ASSERT_EQ(expected, s);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
181 break;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
182 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
183
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
184 default:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
185 throw OrthancException(ErrorCode_InternalError);
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
186 }
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
187 }
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
188
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
189 void CheckNoChild(int64_t id)
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
190 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
191 std::list<std::string> j;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
192
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
193 switch (GetParam())
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
194 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
195 case DatabaseWrapperClass_SQLite:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
196 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
197 DatabaseWrapper* sqlite = dynamic_cast<DatabaseWrapper*>(index_.get());
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
198 sqlite->GetChildren(j, id);
1492
9f66a12eb8fc fix warnings for gcc 5
jodogne
parents: 1486
diff changeset
199 ASSERT_EQ(0u, j.size());
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
200 break;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
201 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
202
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
203 default:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
204 throw OrthancException(ErrorCode_InternalError);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
205 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
206 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
207
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
208 void CheckOneChild(const char* expected, int64_t id)
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
209 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
210 std::list<std::string> j;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
211
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
212 switch (GetParam())
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
213 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
214 case DatabaseWrapperClass_SQLite:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
215 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
216 DatabaseWrapper* sqlite = dynamic_cast<DatabaseWrapper*>(index_.get());
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
217 sqlite->GetChildren(j, id);
1492
9f66a12eb8fc fix warnings for gcc 5
jodogne
parents: 1486
diff changeset
218 ASSERT_EQ(1u, j.size());
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
219 ASSERT_EQ(expected, j.front());
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
220 break;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
221 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
222
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
223 default:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
224 throw OrthancException(ErrorCode_InternalError);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
225 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
226 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
227
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
228 void CheckTwoChildren(const char* expected1,
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
229 const char* expected2,
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
230 int64_t id)
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
231 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
232 std::list<std::string> j;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
233
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
234 switch (GetParam())
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
235 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
236 case DatabaseWrapperClass_SQLite:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
237 {
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
238 DatabaseWrapper* sqlite = dynamic_cast<DatabaseWrapper*>(index_.get());
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
239 sqlite->GetChildren(j, id);
1492
9f66a12eb8fc fix warnings for gcc 5
jodogne
parents: 1486
diff changeset
240 ASSERT_EQ(2u, j.size());
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
241 ASSERT_TRUE((expected1 == j.front() && expected2 == j.back()) ||
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
242 (expected1 == j.back() && expected2 == j.front()));
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
243 break;
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
244 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
245
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
246 default:
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
247 throw OrthancException(ErrorCode_InternalError);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
248 }
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
249 }
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
250
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
251
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
252 void DoLookup(std::list<std::string>& result,
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
253 ResourceType level,
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
254 const DicomTag& tag,
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
255 const std::string& value)
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
256 {
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
257 LookupIdentifierQuery query(level);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
258 query.AddConstraint(tag, IdentifierConstraintType_Equal, value);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
259 query.Apply(result, *index_);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
260 }
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
261
737
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
262 };
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
263 }
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
264
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
265
737
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
266 INSTANTIATE_TEST_CASE_P(DatabaseWrapperName,
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
267 DatabaseWrapperTest,
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
268 ::testing::Values(DatabaseWrapperClass_SQLite));
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
269
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
270
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
271 TEST_P(DatabaseWrapperTest, Simple)
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
272 {
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
273 int64_t a[] = {
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
274 index_->CreateResource("a", ResourceType_Patient), // 0
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
275 index_->CreateResource("b", ResourceType_Study), // 1
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
276 index_->CreateResource("c", ResourceType_Series), // 2
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
277 index_->CreateResource("d", ResourceType_Instance), // 3
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
278 index_->CreateResource("e", ResourceType_Instance), // 4
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
279 index_->CreateResource("f", ResourceType_Instance), // 5
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
280 index_->CreateResource("g", ResourceType_Study) // 6
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
281 };
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
282
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
283 ASSERT_EQ("a", index_->GetPublicId(a[0]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
284 ASSERT_EQ("b", index_->GetPublicId(a[1]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
285 ASSERT_EQ("c", index_->GetPublicId(a[2]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
286 ASSERT_EQ("d", index_->GetPublicId(a[3]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
287 ASSERT_EQ("e", index_->GetPublicId(a[4]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
288 ASSERT_EQ("f", index_->GetPublicId(a[5]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
289 ASSERT_EQ("g", index_->GetPublicId(a[6]));
198
663cc6c46d0a before refactoring of ServerIndex::GetXXX
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 197
diff changeset
290
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
291 ASSERT_EQ(ResourceType_Patient, index_->GetResourceType(a[0]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
292 ASSERT_EQ(ResourceType_Study, index_->GetResourceType(a[1]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
293 ASSERT_EQ(ResourceType_Series, index_->GetResourceType(a[2]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
294 ASSERT_EQ(ResourceType_Instance, index_->GetResourceType(a[3]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
295 ASSERT_EQ(ResourceType_Instance, index_->GetResourceType(a[4]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
296 ASSERT_EQ(ResourceType_Instance, index_->GetResourceType(a[5]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
297 ASSERT_EQ(ResourceType_Study, index_->GetResourceType(a[6]));
304
4eea080e6e7a refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 273
diff changeset
298
190
b6cef9d45cc3 getallpublicids
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 188
diff changeset
299 {
1241
90d2f320862d refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1239
diff changeset
300 std::list<std::string> t;
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
301 index_->GetAllPublicIds(t, ResourceType_Patient);
190
b6cef9d45cc3 getallpublicids
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 188
diff changeset
302
232
5368bbe813cf refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 218
diff changeset
303 ASSERT_EQ(1u, t.size());
1241
90d2f320862d refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1239
diff changeset
304 ASSERT_EQ("a", t.front());
190
b6cef9d45cc3 getallpublicids
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 188
diff changeset
305
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
306 index_->GetAllPublicIds(t, ResourceType_Series);
232
5368bbe813cf refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 218
diff changeset
307 ASSERT_EQ(1u, t.size());
1241
90d2f320862d refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1239
diff changeset
308 ASSERT_EQ("c", t.front());
190
b6cef9d45cc3 getallpublicids
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 188
diff changeset
309
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
310 index_->GetAllPublicIds(t, ResourceType_Study);
232
5368bbe813cf refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 218
diff changeset
311 ASSERT_EQ(2u, t.size());
190
b6cef9d45cc3 getallpublicids
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 188
diff changeset
312
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
313 index_->GetAllPublicIds(t, ResourceType_Instance);
232
5368bbe813cf refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 218
diff changeset
314 ASSERT_EQ(3u, t.size());
190
b6cef9d45cc3 getallpublicids
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 188
diff changeset
315 }
b6cef9d45cc3 getallpublicids
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 188
diff changeset
316
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
317 index_->SetGlobalProperty(GlobalProperty_FlushSleep, "World");
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
318
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
319 index_->AttachChild(a[0], a[1]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
320 index_->AttachChild(a[1], a[2]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
321 index_->AttachChild(a[2], a[3]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
322 index_->AttachChild(a[2], a[4]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
323 index_->AttachChild(a[6], a[5]);
182
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
324
198
663cc6c46d0a before refactoring of ServerIndex::GetXXX
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 197
diff changeset
325 int64_t parent;
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
326 ASSERT_FALSE(index_->LookupParent(parent, a[0]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
327 ASSERT_TRUE(index_->LookupParent(parent, a[1])); ASSERT_EQ(a[0], parent);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
328 ASSERT_TRUE(index_->LookupParent(parent, a[2])); ASSERT_EQ(a[1], parent);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
329 ASSERT_TRUE(index_->LookupParent(parent, a[3])); ASSERT_EQ(a[2], parent);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
330 ASSERT_TRUE(index_->LookupParent(parent, a[4])); ASSERT_EQ(a[2], parent);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
331 ASSERT_TRUE(index_->LookupParent(parent, a[5])); ASSERT_EQ(a[6], parent);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
332 ASSERT_FALSE(index_->LookupParent(parent, a[6]));
198
663cc6c46d0a before refactoring of ServerIndex::GetXXX
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 197
diff changeset
333
182
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
334 std::string s;
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
335
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
336 CheckNoParent(a[0]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
337 CheckNoParent(a[6]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
338 CheckParentPublicId("a", a[1]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
339 CheckParentPublicId("b", a[2]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
340 CheckParentPublicId("c", a[3]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
341 CheckParentPublicId("c", a[4]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
342 CheckParentPublicId("g", a[5]);
182
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
343
185
5185ae1d9af5 fix for msvc
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 184
diff changeset
344 std::list<std::string> l;
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
345 index_->GetChildrenPublicId(l, a[0]); ASSERT_EQ(1u, l.size()); ASSERT_EQ("b", l.front());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
346 index_->GetChildrenPublicId(l, a[1]); ASSERT_EQ(1u, l.size()); ASSERT_EQ("c", l.front());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
347 index_->GetChildrenPublicId(l, a[3]); ASSERT_EQ(0u, l.size());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
348 index_->GetChildrenPublicId(l, a[4]); ASSERT_EQ(0u, l.size());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
349 index_->GetChildrenPublicId(l, a[5]); ASSERT_EQ(0u, l.size());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
350 index_->GetChildrenPublicId(l, a[6]); ASSERT_EQ(1u, l.size()); ASSERT_EQ("f", l.front());
182
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
351
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
352 index_->GetChildrenPublicId(l, a[2]); ASSERT_EQ(2u, l.size());
182
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
353 if (l.front() == "d")
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
354 {
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
355 ASSERT_EQ("e", l.back());
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
356 }
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
357 else
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
358 {
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
359 ASSERT_EQ("d", l.back());
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
360 ASSERT_EQ("e", l.front());
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
361 }
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
362
436
d51186bf7602 read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 310
diff changeset
363 std::list<MetadataType> md;
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
364 index_->ListAvailableMetadata(md, a[4]);
436
d51186bf7602 read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 310
diff changeset
365 ASSERT_EQ(0u, md.size());
d51186bf7602 read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 310
diff changeset
366
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
367 index_->AddAttachment(a[4], FileInfo("my json file", FileContentType_DicomAsJson, 42, "md5",
1514
d73a2178b319 support of deflate and gzip content-types
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1492
diff changeset
368 CompressionType_ZlibWithSize, 21, "compressedMD5"));
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
369 index_->AddAttachment(a[4], FileInfo("my dicom file", FileContentType_Dicom, 42, "md5"));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
370 index_->AddAttachment(a[6], FileInfo("world", FileContentType_Dicom, 44, "md5"));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
371 index_->SetMetadata(a[4], MetadataType_Instance_RemoteAet, "PINNACLE");
436
d51186bf7602 read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 310
diff changeset
372
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
373 index_->ListAvailableMetadata(md, a[4]);
436
d51186bf7602 read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 310
diff changeset
374 ASSERT_EQ(1u, md.size());
d51186bf7602 read access to metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 310
diff changeset
375 ASSERT_EQ(MetadataType_Instance_RemoteAet, md.front());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
376 index_->SetMetadata(a[4], MetadataType_ModifiedFrom, "TUTU");
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
377 index_->ListAvailableMetadata(md, a[4]);
438
7bbe77cb9e12 DELETE metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 436
diff changeset
378 ASSERT_EQ(2u, md.size());
1030
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
379
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
380 std::map<MetadataType, std::string> md2;
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
381 index_->GetAllMetadata(md2, a[4]);
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
382 ASSERT_EQ(2u, md2.size());
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
383 ASSERT_EQ("TUTU", md2[MetadataType_ModifiedFrom]);
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
384 ASSERT_EQ("PINNACLE", md2[MetadataType_Instance_RemoteAet]);
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
385
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
386 index_->DeleteMetadata(a[4], MetadataType_ModifiedFrom);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
387 index_->ListAvailableMetadata(md, a[4]);
438
7bbe77cb9e12 DELETE metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 436
diff changeset
388 ASSERT_EQ(1u, md.size());
7bbe77cb9e12 DELETE metadata
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 436
diff changeset
389 ASSERT_EQ(MetadataType_Instance_RemoteAet, md.front());
182
93ff5babcaf8 children public id
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 181
diff changeset
390
1030
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
391 index_->GetAllMetadata(md2, a[4]);
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
392 ASSERT_EQ(1u, md2.size());
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
393 ASSERT_EQ("PINNACLE", md2[MetadataType_Instance_RemoteAet]);
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
394
66fa947c7280 more tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1029
diff changeset
395
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
396 ASSERT_EQ(21u + 42u + 44u, index_->GetTotalCompressedSize());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
397 ASSERT_EQ(42u + 42u + 44u, index_->GetTotalUncompressedSize());
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
398
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
399 index_->SetMainDicomTag(a[3], DicomTag(0x0010, 0x0010), "PatientName");
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
400
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
401 int64_t b;
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
402 ResourceType t;
1294
910478b2d4e4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1288
diff changeset
403 ASSERT_TRUE(index_->LookupResource(b, t, "g"));
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
404 ASSERT_EQ(7, b);
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
405 ASSERT_EQ(ResourceType_Study, t);
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
406
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
407 ASSERT_TRUE(index_->LookupMetadata(s, a[4], MetadataType_Instance_RemoteAet));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
408 ASSERT_FALSE(index_->LookupMetadata(s, a[4], MetadataType_Instance_IndexInSeries));
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
409 ASSERT_EQ("PINNACLE", s);
1238
6c07108ff1e2 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1237
diff changeset
410
6c07108ff1e2 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1237
diff changeset
411 std::string u;
6c07108ff1e2 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1237
diff changeset
412 ASSERT_TRUE(index_->LookupMetadata(u, a[4], MetadataType_Instance_RemoteAet));
6c07108ff1e2 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1237
diff changeset
413 ASSERT_EQ("PINNACLE", u);
6c07108ff1e2 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1237
diff changeset
414 ASSERT_FALSE(index_->LookupMetadata(u, a[4], MetadataType_Instance_IndexInSeries));
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
415
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
416 ASSERT_TRUE(index_->LookupGlobalProperty(s, GlobalProperty_FlushSleep));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
417 ASSERT_FALSE(index_->LookupGlobalProperty(s, static_cast<GlobalProperty>(42)));
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
418 ASSERT_EQ("World", s);
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
419
232
5368bbe813cf refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 218
diff changeset
420 FileInfo att;
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
421 ASSERT_TRUE(index_->LookupAttachment(att, a[4], FileContentType_DicomAsJson));
232
5368bbe813cf refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 218
diff changeset
422 ASSERT_EQ("my json file", att.GetUuid());
5368bbe813cf refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 218
diff changeset
423 ASSERT_EQ(21u, att.GetCompressedSize());
693
01d8611c4a60 md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 632
diff changeset
424 ASSERT_EQ("md5", att.GetUncompressedMD5());
01d8611c4a60 md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 632
diff changeset
425 ASSERT_EQ("compressedMD5", att.GetCompressedMD5());
232
5368bbe813cf refactoring of attachments
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 218
diff changeset
426 ASSERT_EQ(42u, att.GetUncompressedSize());
1514
d73a2178b319 support of deflate and gzip content-types
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1492
diff changeset
427 ASSERT_EQ(CompressionType_ZlibWithSize, att.GetCompressionType());
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
428
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
429 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
430 ASSERT_EQ("world", att.GetUuid());
01d8611c4a60 md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 632
diff changeset
431 ASSERT_EQ(44u, att.GetCompressedSize());
01d8611c4a60 md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 632
diff changeset
432 ASSERT_EQ("md5", att.GetUncompressedMD5());
01d8611c4a60 md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 632
diff changeset
433 ASSERT_EQ("md5", att.GetCompressedMD5());
01d8611c4a60 md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 632
diff changeset
434 ASSERT_EQ(44u, att.GetUncompressedSize());
01d8611c4a60 md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 632
diff changeset
435 ASSERT_EQ(CompressionType_None, att.GetCompressionType());
01d8611c4a60 md5 for attached files
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 632
diff changeset
436
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
437 ASSERT_EQ(0u, listener_->deletedFiles_.size());
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
438 ASSERT_EQ(0u, listener_->deletedResources_.size());
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
439
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
440 CheckTableRecordCount(7, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
441 CheckTableRecordCount(3, "AttachedFiles");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
442 CheckTableRecordCount(1, "Metadata");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
443 CheckTableRecordCount(1, "MainDicomTags");
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
444
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
445 index_->DeleteResource(a[0]);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
446 ASSERT_EQ(5u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
447 ASSERT_EQ(2u, listener_->deletedFiles_.size());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
448 ASSERT_FALSE(std::find(listener_->deletedFiles_.begin(),
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
449 listener_->deletedFiles_.end(),
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
450 "my json file") == listener_->deletedFiles_.end());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
451 ASSERT_FALSE(std::find(listener_->deletedFiles_.begin(),
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
452 listener_->deletedFiles_.end(),
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
453 "my dicom file") == listener_->deletedFiles_.end());
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
454
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
455 CheckTableRecordCount(2, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
456 CheckTableRecordCount(0, "Metadata");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
457 CheckTableRecordCount(1, "AttachedFiles");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
458 CheckTableRecordCount(0, "MainDicomTags");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
459
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
460 index_->DeleteResource(a[5]);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
461 ASSERT_EQ(7u, listener_->deletedResources_.size());
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
462
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
463 CheckTableRecordCount(0, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
464 CheckTableRecordCount(0, "AttachedFiles");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
465 CheckTableRecordCount(2, "GlobalProperties");
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
466
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
467 ASSERT_EQ(3u, listener_->deletedFiles_.size());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
468 ASSERT_FALSE(std::find(listener_->deletedFiles_.begin(),
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
469 listener_->deletedFiles_.end(),
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
470 "world") == listener_->deletedFiles_.end());
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
471 }
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
472
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
473
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
474
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
475
737
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
476 TEST_P(DatabaseWrapperTest, Upward)
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
477 {
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
478 int64_t a[] = {
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
479 index_->CreateResource("a", ResourceType_Patient), // 0
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
480 index_->CreateResource("b", ResourceType_Study), // 1
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
481 index_->CreateResource("c", ResourceType_Series), // 2
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
482 index_->CreateResource("d", ResourceType_Instance), // 3
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
483 index_->CreateResource("e", ResourceType_Instance), // 4
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
484 index_->CreateResource("f", ResourceType_Study), // 5
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
485 index_->CreateResource("g", ResourceType_Series), // 6
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
486 index_->CreateResource("h", ResourceType_Series) // 7
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
487 };
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
488
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
489 index_->AttachChild(a[0], a[1]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
490 index_->AttachChild(a[1], a[2]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
491 index_->AttachChild(a[2], a[3]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
492 index_->AttachChild(a[2], a[4]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
493 index_->AttachChild(a[1], a[6]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
494 index_->AttachChild(a[0], a[5]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
495 index_->AttachChild(a[5], a[7]);
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
496
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
497 CheckTwoChildren("b", "f", a[0]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
498 CheckTwoChildren("c", "g", a[1]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
499 CheckTwoChildren("d", "e", a[2]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
500 CheckNoChild(a[3]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
501 CheckNoChild(a[4]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
502 CheckOneChild("h", a[5]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
503 CheckNoChild(a[6]);
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
504 CheckNoChild(a[7]);
193
a1b9d1e1497b failed attempt to compile with linux standard base
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 190
diff changeset
505
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
506 listener_->Reset();
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
507 index_->DeleteResource(a[3]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
508 ASSERT_EQ("c", listener_->ancestorId_);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
509 ASSERT_EQ(ResourceType_Series, listener_->ancestorType_);
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
510
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
511 listener_->Reset();
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
512 index_->DeleteResource(a[4]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
513 ASSERT_EQ("b", listener_->ancestorId_);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
514 ASSERT_EQ(ResourceType_Study, listener_->ancestorType_);
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
515
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
516 listener_->Reset();
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
517 index_->DeleteResource(a[7]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
518 ASSERT_EQ("a", listener_->ancestorId_);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
519 ASSERT_EQ(ResourceType_Patient, listener_->ancestorType_);
183
baada606da3c databasewrapper
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 182
diff changeset
520
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
521 listener_->Reset();
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
522 index_->DeleteResource(a[6]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
523 ASSERT_EQ("", listener_->ancestorId_); // No more ancestor
181
2dece1526c06 simplifying db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
524 }
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
525
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
526
737
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
527 TEST_P(DatabaseWrapperTest, PatientRecycling)
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
528 {
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
529 std::vector<int64_t> patients;
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
530 for (int i = 0; i < 10; i++)
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
531 {
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
532 std::string p = "Patient " + boost::lexical_cast<std::string>(i);
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
533 patients.push_back(index_->CreateResource(p, ResourceType_Patient));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
534 index_->AddAttachment(patients[i], FileInfo(p, FileContentType_Dicom, i + 10,
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
535 "md5-" + boost::lexical_cast<std::string>(i)));
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
536 ASSERT_FALSE(index_->IsProtectedPatient(patients[i]));
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
537 }
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
538
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
539 CheckTableRecordCount(10u, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
540 CheckTableRecordCount(10u, "PatientRecyclingOrder");
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
541
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
542 listener_->Reset();
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
543 ASSERT_EQ(0u, listener_->deletedResources_.size());
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
544
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
545 index_->DeleteResource(patients[5]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
546 index_->DeleteResource(patients[0]);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
547 ASSERT_EQ(2u, listener_->deletedResources_.size());
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
548
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
549 CheckTableRecordCount(8u, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
550 CheckTableRecordCount(8u, "PatientRecyclingOrder");
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
551
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
552 ASSERT_EQ(2u, listener_->deletedFiles_.size());
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
553 ASSERT_EQ("Patient 5", listener_->deletedFiles_[0]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
554 ASSERT_EQ("Patient 0", listener_->deletedFiles_[1]);
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
555
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
556 int64_t p;
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
557 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[1]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
558 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
559 ASSERT_EQ(3u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
560 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[2]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
561 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
562 ASSERT_EQ(4u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
563 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[3]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
564 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
565 ASSERT_EQ(5u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
566 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[4]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
567 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
568 ASSERT_EQ(6u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
569 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[6]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
570 index_->DeleteResource(p);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
571 index_->DeleteResource(patients[8]);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
572 ASSERT_EQ(8u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
573 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[7]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
574 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
575 ASSERT_EQ(9u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
576 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[9]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
577 index_->DeleteResource(p);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
578 ASSERT_FALSE(index_->SelectPatientToRecycle(p));
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
579 ASSERT_EQ(10u, listener_->deletedResources_.size());
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
580
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
581 ASSERT_EQ(10u, listener_->deletedFiles_.size());
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
582
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
583 CheckTableRecordCount(0, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
584 CheckTableRecordCount(0, "PatientRecyclingOrder");
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
585 }
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
586
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
587
737
1dee6e9bdbf4 abstraction of databasewrapper tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 726
diff changeset
588 TEST_P(DatabaseWrapperTest, PatientProtection)
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
589 {
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
590 std::vector<int64_t> patients;
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
591 for (int i = 0; i < 5; i++)
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
592 {
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
593 std::string p = "Patient " + boost::lexical_cast<std::string>(i);
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
594 patients.push_back(index_->CreateResource(p, ResourceType_Patient));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
595 index_->AddAttachment(patients[i], FileInfo(p, FileContentType_Dicom, i + 10,
1287
63a6428771f4 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1286
diff changeset
596 "md5-" + boost::lexical_cast<std::string>(i)));
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
597 ASSERT_FALSE(index_->IsProtectedPatient(patients[i]));
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
598 }
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
599
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
600 CheckTableRecordCount(5, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
601 CheckTableRecordCount(5, "PatientRecyclingOrder");
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
602
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
603 ASSERT_FALSE(index_->IsProtectedPatient(patients[2]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
604 index_->SetProtectedPatient(patients[2], true);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
605 ASSERT_TRUE(index_->IsProtectedPatient(patients[2]));
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
606 CheckTableRecordCount(5, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
607 CheckTableRecordCount(4, "PatientRecyclingOrder");
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
608
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
609 index_->SetProtectedPatient(patients[2], true);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
610 ASSERT_TRUE(index_->IsProtectedPatient(patients[2]));
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
611 CheckTableRecordCount(4, "PatientRecyclingOrder");
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
612 index_->SetProtectedPatient(patients[2], false);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
613 ASSERT_FALSE(index_->IsProtectedPatient(patients[2]));
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
614 CheckTableRecordCount(5, "PatientRecyclingOrder");
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
615 index_->SetProtectedPatient(patients[2], false);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
616 ASSERT_FALSE(index_->IsProtectedPatient(patients[2]));
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
617 CheckTableRecordCount(5, "PatientRecyclingOrder");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
618 CheckTableRecordCount(5, "Resources");
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
619 index_->SetProtectedPatient(patients[2], true);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
620 ASSERT_TRUE(index_->IsProtectedPatient(patients[2]));
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
621 CheckTableRecordCount(4, "PatientRecyclingOrder");
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
622 index_->SetProtectedPatient(patients[2], false);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
623 ASSERT_FALSE(index_->IsProtectedPatient(patients[2]));
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
624 CheckTableRecordCount(5, "PatientRecyclingOrder");
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
625 index_->SetProtectedPatient(patients[3], true);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
626 ASSERT_TRUE(index_->IsProtectedPatient(patients[3]));
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
627 CheckTableRecordCount(4, "PatientRecyclingOrder");
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
628
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
629 CheckTableRecordCount(5, "Resources");
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
630 ASSERT_EQ(0u, listener_->deletedFiles_.size());
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
631
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
632 // 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
633 int64_t p;
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
634 ASSERT_EQ(0u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
635 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[0]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
636 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
637 ASSERT_EQ(1u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
638 ASSERT_TRUE(index_->SelectPatientToRecycle(p, patients[1])); ASSERT_EQ(p, patients[4]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
639 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[1]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
640 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
641 ASSERT_EQ(2u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
642 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[4]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
643 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
644 ASSERT_EQ(3u, listener_->deletedResources_.size());
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
645 ASSERT_FALSE(index_->SelectPatientToRecycle(p, patients[2]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
646 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[2]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
647 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
648 ASSERT_EQ(4u, listener_->deletedResources_.size());
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
649 // "patients[3]" is still protected
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
650 ASSERT_FALSE(index_->SelectPatientToRecycle(p));
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
651
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
652 ASSERT_EQ(4u, listener_->deletedFiles_.size());
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
653 CheckTableRecordCount(1, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
654 CheckTableRecordCount(0, "PatientRecyclingOrder");
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
655
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
656 index_->SetProtectedPatient(patients[3], false);
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
657 CheckTableRecordCount(1, "PatientRecyclingOrder");
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
658 ASSERT_FALSE(index_->SelectPatientToRecycle(p, patients[3]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
659 ASSERT_TRUE(index_->SelectPatientToRecycle(p, patients[2]));
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
660 ASSERT_TRUE(index_->SelectPatientToRecycle(p)); ASSERT_EQ(p, patients[3]);
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
661 index_->DeleteResource(p);
1159
8f9d49192815 speeding up db schema
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1134
diff changeset
662 ASSERT_EQ(5u, listener_->deletedResources_.size());
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
663
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
664 ASSERT_EQ(5u, listener_->deletedFiles_.size());
1286
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
665 CheckTableRecordCount(0, "Resources");
b4acdb37e43b refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1252
diff changeset
666 CheckTableRecordCount(0, "PatientRecyclingOrder");
262
2354560daf2f primitives for recycling patients
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 252
diff changeset
667 }
310
6ab6cdeedf4e global sequences
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 304
diff changeset
668
6ab6cdeedf4e global sequences
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 304
diff changeset
669
6ab6cdeedf4e global sequences
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 304
diff changeset
670
1237
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
671 TEST(ServerIndex, Sequence)
310
6ab6cdeedf4e global sequences
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 304
diff changeset
672 {
1237
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
673 const std::string path = "UnitTestsStorage";
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
674
2140
aa4b8895cd23 reorganization
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2121
diff changeset
675 SystemToolbox::RemoveFile(path + "/index");
1237
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
676 FilesystemStorage storage(path);
1247
32fcc5dc7562 abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1241
diff changeset
677 DatabaseWrapper db; // The SQLite DB is in memory
1672
4c5a85c3ff43 sample database plugin now working
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1549
diff changeset
678 db.Open();
2815
925d8dc03a23 unserialization of jobs from plugins
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2671
diff changeset
679 ServerContext context(db, storage, true /* running unit tests */);
925d8dc03a23 unserialization of jobs from plugins
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2671
diff changeset
680 context.SetupJobsEngine(true, false);
925d8dc03a23 unserialization of jobs from plugins
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2671
diff changeset
681
1237
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
682 ServerIndex& index = context.GetIndex();
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
683
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
684 ASSERT_EQ(1u, index.IncrementGlobalSequence(GlobalProperty_AnonymizationSequence));
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
685 ASSERT_EQ(2u, index.IncrementGlobalSequence(GlobalProperty_AnonymizationSequence));
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
686 ASSERT_EQ(3u, index.IncrementGlobalSequence(GlobalProperty_AnonymizationSequence));
0f3716b88af7 cleaning
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1236
diff changeset
687 ASSERT_EQ(4u, index.IncrementGlobalSequence(GlobalProperty_AnonymizationSequence));
1453
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1432
diff changeset
688
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1432
diff changeset
689 context.Stop();
1672
4c5a85c3ff43 sample database plugin now working
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1549
diff changeset
690 db.Close();
310
6ab6cdeedf4e global sequences
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 304
diff changeset
691 }
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
692
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
693
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
694
1162
1ea4094d077c refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1159
diff changeset
695 TEST_P(DatabaseWrapperTest, LookupIdentifier)
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
696 {
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
697 int64_t a[] = {
738
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
698 index_->CreateResource("a", ResourceType_Study), // 0
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
699 index_->CreateResource("b", ResourceType_Study), // 1
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
700 index_->CreateResource("c", ResourceType_Study), // 2
c60743fadd4e more abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 737
diff changeset
701 index_->CreateResource("d", ResourceType_Series) // 3
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
702 };
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
703
1713
4db9200c7f46 SetIdentifierTag
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1672
diff changeset
704 index_->SetIdentifierTag(a[0], DICOM_TAG_STUDY_INSTANCE_UID, "0");
4db9200c7f46 SetIdentifierTag
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1672
diff changeset
705 index_->SetIdentifierTag(a[1], DICOM_TAG_STUDY_INSTANCE_UID, "1");
4db9200c7f46 SetIdentifierTag
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1672
diff changeset
706 index_->SetIdentifierTag(a[2], DICOM_TAG_STUDY_INSTANCE_UID, "0");
4db9200c7f46 SetIdentifierTag
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1672
diff changeset
707 index_->SetIdentifierTag(a[3], DICOM_TAG_SERIES_INSTANCE_UID, "0");
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
708
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
709 std::list<std::string> s;
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
710
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
711 DoLookup(s, ResourceType_Study, DICOM_TAG_STUDY_INSTANCE_UID, "0");
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
712 ASSERT_EQ(2u, s.size());
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
713 ASSERT_TRUE(std::find(s.begin(), s.end(), "a") != s.end());
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
714 ASSERT_TRUE(std::find(s.begin(), s.end(), "c") != s.end());
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
715
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
716 DoLookup(s, ResourceType_Series, DICOM_TAG_SERIES_INSTANCE_UID, "0");
1718
2b812969e136 getting rid of an IDatabaseWrapper::LookupIdentifier flavor
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1713
diff changeset
717 ASSERT_EQ(1u, s.size());
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
718 ASSERT_TRUE(std::find(s.begin(), s.end(), "d") != s.end());
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
719
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
720 DoLookup(s, ResourceType_Study, DICOM_TAG_STUDY_INSTANCE_UID, "1");
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
721 ASSERT_EQ(1u, s.size());
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
722 ASSERT_TRUE(std::find(s.begin(), s.end(), "b") != s.end());
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
723
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
724 DoLookup(s, ResourceType_Study, DICOM_TAG_STUDY_INSTANCE_UID, "1");
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
725 ASSERT_EQ(1u, s.size());
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
726 ASSERT_TRUE(std::find(s.begin(), s.end(), "b") != s.end());
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
727
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
728 DoLookup(s, ResourceType_Series, DICOM_TAG_SERIES_INSTANCE_UID, "1");
1718
2b812969e136 getting rid of an IDatabaseWrapper::LookupIdentifier flavor
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1713
diff changeset
729 ASSERT_EQ(0u, s.size());
521
2c739f76d0bb lookup tag values
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 438
diff changeset
730
1746
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
731 {
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
732 LookupIdentifierQuery query(ResourceType_Study);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
733 query.AddConstraint(DICOM_TAG_STUDY_INSTANCE_UID, IdentifierConstraintType_GreaterOrEqual, "0");
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
734 query.Apply(s, *index_);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
735 ASSERT_EQ(3u, s.size());
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
736 }
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
737
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
738 {
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
739 LookupIdentifierQuery query(ResourceType_Study);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
740 query.AddConstraint(DICOM_TAG_STUDY_INSTANCE_UID, IdentifierConstraintType_GreaterOrEqual, "0");
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
741 query.AddConstraint(DICOM_TAG_STUDY_INSTANCE_UID, IdentifierConstraintType_SmallerOrEqual, "0");
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
742 query.Apply(s, *index_);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
743 ASSERT_EQ(2u, s.size());
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
744 }
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
745
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
746 {
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
747 LookupIdentifierQuery query(ResourceType_Study);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
748 query.AddConstraint(DICOM_TAG_STUDY_INSTANCE_UID, IdentifierConstraintType_GreaterOrEqual, "1");
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
749 query.AddConstraint(DICOM_TAG_STUDY_INSTANCE_UID, IdentifierConstraintType_SmallerOrEqual, "1");
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
750 query.Apply(s, *index_);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
751 ASSERT_EQ(1u, s.size());
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
752 }
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
753
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
754 {
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
755 LookupIdentifierQuery query(ResourceType_Study);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
756 query.AddConstraint(DICOM_TAG_STUDY_INSTANCE_UID, IdentifierConstraintType_GreaterOrEqual, "1");
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
757 query.Apply(s, *index_);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
758 ASSERT_EQ(1u, s.size());
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
759 }
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
760
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
761 {
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
762 LookupIdentifierQuery query(ResourceType_Study);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
763 query.AddConstraint(DICOM_TAG_STUDY_INSTANCE_UID, IdentifierConstraintType_GreaterOrEqual, "2");
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
764 query.Apply(s, *index_);
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
765 ASSERT_EQ(0u, s.size());
d143db00a794 SetOfResources
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1744
diff changeset
766 }
562
f64e3838d6e1 refactoring enumerations
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 521
diff changeset
767 }
705
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
768
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
769
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
770
713
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
771 TEST(ServerIndex, AttachmentRecycling)
705
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
772 {
803
4689e400e0fa directory to store the results of the unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 738
diff changeset
773 const std::string path = "UnitTestsStorage";
1124
790ff7a5b3bf IStorageArea abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1030
diff changeset
774
2140
aa4b8895cd23 reorganization
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2121
diff changeset
775 SystemToolbox::RemoveFile(path + "/index");
1124
790ff7a5b3bf IStorageArea abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1030
diff changeset
776 FilesystemStorage storage(path);
1247
32fcc5dc7562 abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1241
diff changeset
777 DatabaseWrapper db; // The SQLite DB is in memory
1672
4c5a85c3ff43 sample database plugin now working
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1549
diff changeset
778 db.Open();
2815
925d8dc03a23 unserialization of jobs from plugins
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2671
diff changeset
779 ServerContext context(db, storage, true /* running unit tests */);
925d8dc03a23 unserialization of jobs from plugins
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2671
diff changeset
780 context.SetupJobsEngine(true, false);
705
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
781 ServerIndex& index = context.GetIndex();
707
203157cb4fde unit tests of httpclient
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 706
diff changeset
782
706
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 705
diff changeset
783 index.SetMaximumStorageSize(10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 705
diff changeset
784
705
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
785 Json::Value tmp;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
786 index.ComputeStatistics(tmp);
713
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
787 ASSERT_EQ(0, tmp["CountPatients"].asInt());
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
788 ASSERT_EQ(0, boost::lexical_cast<int>(tmp["TotalDiskSize"].asString()));
705
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
789
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
790 ServerIndex::Attachments attachments;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
791
713
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
792 std::vector<std::string> ids;
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
793 for (int i = 0; i < 10; i++)
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
794 {
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
795 std::string id = boost::lexical_cast<std::string>(i);
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
796 DicomMap instance;
2007
655489d9165d DicomMap::ParseDicomMetaInformation()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1971
diff changeset
797 instance.SetValue(DICOM_TAG_PATIENT_ID, "patient-" + id, false);
655489d9165d DicomMap::ParseDicomMetaInformation()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1971
diff changeset
798 instance.SetValue(DICOM_TAG_STUDY_INSTANCE_UID, "study-" + id, false);
655489d9165d DicomMap::ParseDicomMetaInformation()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1971
diff changeset
799 instance.SetValue(DICOM_TAG_SERIES_INSTANCE_UID, "series-" + id, false);
655489d9165d DicomMap::ParseDicomMetaInformation()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1971
diff changeset
800 instance.SetValue(DICOM_TAG_SOP_INSTANCE_UID, "instance-" + id, false);
2208
90ea60bee5ff New metadata automatically computed at the instance level: "SopClassUid"
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2190
diff changeset
801 instance.SetValue(DICOM_TAG_SOP_CLASS_UID, "1.2.840.10008.5.1.4.1.1.1", false); // CR image
1006
649d47854314 proper handling of metadata in Store
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 831
diff changeset
802
649d47854314 proper handling of metadata in Store
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 831
diff changeset
803 std::map<MetadataType, std::string> instanceMetadata;
1822
9ed9458aa44f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1764
diff changeset
804 DicomInstanceToStore toStore;
9ed9458aa44f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1764
diff changeset
805 toStore.SetSummary(instance);
9ed9458aa44f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1764
diff changeset
806 ASSERT_EQ(StoreStatus_Success, index.Store(instanceMetadata, toStore, attachments));
2208
90ea60bee5ff New metadata automatically computed at the instance level: "SopClassUid"
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2190
diff changeset
807 ASSERT_EQ(5u, instanceMetadata.size());
1029
9e04f9f36096 fix windows build
jodogne
parents: 1006
diff changeset
808 ASSERT_TRUE(instanceMetadata.find(MetadataType_Instance_RemoteAet) != instanceMetadata.end());
9e04f9f36096 fix windows build
jodogne
parents: 1006
diff changeset
809 ASSERT_TRUE(instanceMetadata.find(MetadataType_Instance_ReceptionDate) != instanceMetadata.end());
2190
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2172
diff changeset
810 ASSERT_TRUE(instanceMetadata.find(MetadataType_Instance_TransferSyntax) != instanceMetadata.end());
2208
90ea60bee5ff New metadata automatically computed at the instance level: "SopClassUid"
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2190
diff changeset
811 ASSERT_TRUE(instanceMetadata.find(MetadataType_Instance_SopClassUid) != instanceMetadata.end());
2190
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2172
diff changeset
812
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2172
diff changeset
813 // By default, an Explicit VR Little Endian is used by Orthanc
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2172
diff changeset
814 ASSERT_EQ("1.2.840.10008.1.2.1", instanceMetadata[MetadataType_Instance_TransferSyntax]);
713
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
815
2208
90ea60bee5ff New metadata automatically computed at the instance level: "SopClassUid"
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2190
diff changeset
816 ASSERT_EQ("1.2.840.10008.5.1.4.1.1.1", instanceMetadata[MetadataType_Instance_SopClassUid]);
90ea60bee5ff New metadata automatically computed at the instance level: "SopClassUid"
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2190
diff changeset
817
713
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
818 DicomInstanceHasher hasher(instance);
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
819 ids.push_back(hasher.HashPatient());
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
820 ids.push_back(hasher.HashStudy());
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
821 ids.push_back(hasher.HashSeries());
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
822 ids.push_back(hasher.HashInstance());
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
823 }
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
824
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
825 index.ComputeStatistics(tmp);
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
826 ASSERT_EQ(10, tmp["CountPatients"].asInt());
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
827 ASSERT_EQ(0, boost::lexical_cast<int>(tmp["TotalDiskSize"].asString()));
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
828
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
829 for (size_t i = 0; i < ids.size(); i++)
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
830 {
2512
4dcafa8d6633 SystemToolbox::GenerateUuid moved to Toolbox::GenerateUuid
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
831 FileInfo info(Toolbox::GenerateUuid(), FileContentType_Dicom, 1, "md5");
713
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
832 index.AddAttachment(info, ids[i]);
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
833
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
834 index.ComputeStatistics(tmp);
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
835 ASSERT_GE(10, boost::lexical_cast<int>(tmp["TotalDiskSize"].asString()));
9d1973813d8b test attachment recycling
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 711
diff changeset
836 }
707
203157cb4fde unit tests of httpclient
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 706
diff changeset
837
203157cb4fde unit tests of httpclient
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 706
diff changeset
838 // Because the DB is in memory, the SQLite index must not have been created
2140
aa4b8895cd23 reorganization
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2121
diff changeset
839 ASSERT_FALSE(SystemToolbox::IsRegularFile(path + "/index"));
1453
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1432
diff changeset
840
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1432
diff changeset
841 context.Stop();
1672
4c5a85c3ff43 sample database plugin now working
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1549
diff changeset
842 db.Close();
705
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 697
diff changeset
843 }
1764
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1747
diff changeset
844
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1747
diff changeset
845
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1747
diff changeset
846 TEST(LookupIdentifierQuery, NormalizeIdentifier)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1747
diff changeset
847 {
2121
7e8889bc95c6 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2007
diff changeset
848 ASSERT_EQ("H^L.LO", ServerToolbox::NormalizeIdentifier(" Hé^l.LO %_ "));
7e8889bc95c6 refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2007
diff changeset
849 ASSERT_EQ("1.2.840.113619.2.176.2025", ServerToolbox::NormalizeIdentifier(" 1.2.840.113619.2.176.2025 "));
1764
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1747
diff changeset
850 }
2826
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
851
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
852
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
853 TEST(ServerIndex, Overwrite)
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
854 {
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
855 for (unsigned int i = 0; i < 2; i++)
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
856 {
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
857 bool overwrite = (i == 0);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
858
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
859 MemoryStorageArea storage;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
860 DatabaseWrapper db; // The SQLite DB is in memory
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
861 db.Open();
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
862 ServerContext context(db, storage, true /* running unit tests */);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
863 context.SetupJobsEngine(true, false);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
864 context.SetCompressionEnabled(true);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
865
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
866 DicomMap instance;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
867 instance.SetValue(DICOM_TAG_PATIENT_ID, "patient", false);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
868 instance.SetValue(DICOM_TAG_PATIENT_NAME, "name", false);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
869 instance.SetValue(DICOM_TAG_STUDY_INSTANCE_UID, "study", false);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
870 instance.SetValue(DICOM_TAG_SERIES_INSTANCE_UID, "series", false);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
871 instance.SetValue(DICOM_TAG_SOP_INSTANCE_UID, "sop", false);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
872 instance.SetValue(DICOM_TAG_SOP_CLASS_UID, "1.2.840.10008.5.1.4.1.1.1", false); // CR image
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
873
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
874 DicomInstanceHasher hasher(instance);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
875 std::string id = hasher.HashInstance();
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
876 context.GetIndex().SetOverwriteInstances(overwrite);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
877
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
878 Json::Value tmp;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
879 context.GetIndex().ComputeStatistics(tmp);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
880 ASSERT_EQ(0, tmp["CountInstances"].asInt());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
881 ASSERT_EQ(0, boost::lexical_cast<int>(tmp["TotalDiskSize"].asString()));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
882
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
883 {
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
884 DicomInstanceToStore toStore;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
885 toStore.SetSummary(instance);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
886 toStore.SetOrigin(DicomInstanceOrigin::FromPlugins());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
887
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
888 std::string id2;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
889 ASSERT_EQ(StoreStatus_Success, context.Store(id2, toStore));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
890 ASSERT_EQ(id, id2);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
891 }
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
892
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
893 FileInfo dicom1, json1;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
894 ASSERT_TRUE(context.GetIndex().LookupAttachment(dicom1, id, FileContentType_Dicom));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
895 ASSERT_TRUE(context.GetIndex().LookupAttachment(json1, id, FileContentType_DicomAsJson));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
896
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
897 context.GetIndex().ComputeStatistics(tmp);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
898 ASSERT_EQ(1, tmp["CountInstances"].asInt());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
899 ASSERT_EQ(dicom1.GetCompressedSize() + json1.GetCompressedSize(),
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
900 boost::lexical_cast<int>(tmp["TotalDiskSize"].asString()));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
901 ASSERT_EQ(dicom1.GetUncompressedSize() + json1.GetUncompressedSize(),
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
902 boost::lexical_cast<int>(tmp["TotalUncompressedSize"].asString()));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
903
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
904 context.ReadDicomAsJson(tmp, id);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
905 ASSERT_EQ("name", tmp["0010,0010"]["Value"].asString());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
906
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
907 {
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
908 ServerContext::DicomCacheLocker locker(context, id);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
909 std::string tmp;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
910 locker.GetDicom().GetTagValue(tmp, DICOM_TAG_PATIENT_NAME);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
911 ASSERT_EQ("name", tmp);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
912 }
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
913
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
914 {
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
915 DicomMap instance2;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
916 instance2.Assign(instance);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
917 instance2.SetValue(DICOM_TAG_PATIENT_NAME, "overwritten", false);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
918
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
919 DicomInstanceToStore toStore;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
920 toStore.SetSummary(instance2);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
921 toStore.SetOrigin(DicomInstanceOrigin::FromPlugins());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
922
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
923 std::string id2;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
924 ASSERT_EQ(overwrite ? StoreStatus_Success : StoreStatus_AlreadyStored, context.Store(id2, toStore));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
925 ASSERT_EQ(id, id2);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
926 }
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
927
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
928 FileInfo dicom2, json2;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
929 ASSERT_TRUE(context.GetIndex().LookupAttachment(dicom2, id, FileContentType_Dicom));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
930 ASSERT_TRUE(context.GetIndex().LookupAttachment(json2, id, FileContentType_DicomAsJson));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
931
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
932 context.GetIndex().ComputeStatistics(tmp);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
933 ASSERT_EQ(1, tmp["CountInstances"].asInt());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
934 ASSERT_EQ(dicom2.GetCompressedSize() + json2.GetCompressedSize(),
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
935 boost::lexical_cast<int>(tmp["TotalDiskSize"].asString()));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
936 ASSERT_EQ(dicom2.GetUncompressedSize() + json2.GetUncompressedSize(),
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
937 boost::lexical_cast<int>(tmp["TotalUncompressedSize"].asString()));
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
938
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
939 if (overwrite)
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
940 {
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
941 ASSERT_NE(dicom1.GetUuid(), dicom2.GetUuid());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
942 ASSERT_NE(json1.GetUuid(), json2.GetUuid());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
943 ASSERT_NE(dicom1.GetUncompressedSize(), dicom2.GetUncompressedSize());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
944 ASSERT_NE(json1.GetUncompressedSize(), json2.GetUncompressedSize());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
945
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
946 context.ReadDicomAsJson(tmp, id);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
947 ASSERT_EQ("overwritten", tmp["0010,0010"]["Value"].asString());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
948
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
949 {
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
950 ServerContext::DicomCacheLocker locker(context, id);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
951 std::string tmp;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
952 locker.GetDicom().GetTagValue(tmp, DICOM_TAG_PATIENT_NAME);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
953 ASSERT_EQ("overwritten", tmp);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
954 }
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
955 }
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
956 else
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
957 {
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
958 ASSERT_EQ(dicom1.GetUuid(), dicom2.GetUuid());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
959 ASSERT_EQ(json1.GetUuid(), json2.GetUuid());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
960 ASSERT_EQ(dicom1.GetUncompressedSize(), dicom2.GetUncompressedSize());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
961 ASSERT_EQ(json1.GetUncompressedSize(), json2.GetUncompressedSize());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
962
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
963 context.ReadDicomAsJson(tmp, id);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
964 ASSERT_EQ("name", tmp["0010,0010"]["Value"].asString());
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
965
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
966 {
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
967 ServerContext::DicomCacheLocker locker(context, id);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
968 std::string tmp;
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
969 locker.GetDicom().GetTagValue(tmp, DICOM_TAG_PATIENT_NAME);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
970 ASSERT_EQ("name", tmp);
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
971 }
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
972 }
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
973
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
974 context.Stop();
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
975 db.Close();
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
976 }
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
977 }
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
978
c277e0421200 unit testing of overwriting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2815
diff changeset
979