Skip to content

Commit

Permalink
Fix server manager version string
Browse files Browse the repository at this point in the history
  • Loading branch information
Diego Nehab committed Jul 8, 2022
1 parent 76217e4 commit 6b8acae
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

- Fix Lua bind of static methods for remote machines
- Fix rollup-memory-range decoding of empty payloads at end of file
- Fix server-manager version string

### Changed

Expand Down
2 changes: 1 addition & 1 deletion src/server-manager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@
#pragma GCC diagnostic pop

static constexpr uint32_t manager_version_major = 0;
static constexpr uint32_t manager_version_minor = 2;
static constexpr uint32_t manager_version_minor = 3;
static constexpr uint32_t manager_version_patch = 0;
static constexpr const char *manager_version_pre_release = "";
static constexpr const char *manager_version_build = "";
Expand Down
4 changes: 2 additions & 2 deletions src/test-server-manager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -665,12 +665,12 @@ void assert_bool(bool value, const std::string &msg, const std::string &file, in
#define ASSERT_STATUS_CODE(s, f, v) assert_status_code(s, f, v, __FILE__, __LINE__)

static void test_get_version(const std::function<void(const std::string &title, test_function f)> &test) {
test("The rollup-machine-manager server version should be 0.2.x", [](ServerManagerClient &manager) {
test("The rollup-machine-manager server version should be 0.3.x", [](ServerManagerClient &manager) {
Versioning::GetVersionResponse response;
Status status = manager.get_version(response);
ASSERT_STATUS(status, "GetVersion", true);
ASSERT((response.version().major() == 0), "Version Major should be 0");
ASSERT((response.version().minor() == 2), "Version Minor should be 2");
ASSERT((response.version().minor() == 3), "Version Minor should be 3");
});
}

Expand Down

0 comments on commit 6b8acae

Please sign in to comment.