Skip to content

Map server tests#1

Merged
bpwilcox merged 5 commits intomap_server-develfrom
map_server_tests
Aug 30, 2018
Merged

Map server tests#1
bpwilcox merged 5 commits intomap_server-develfrom
map_server_tests

Conversation

@bpwilcox
Copy link
Owner

changing working directory for tests back to map_server-devel

@bpwilcox bpwilcox merged commit a865d1c into map_server-devel Aug 30, 2018
@bpwilcox bpwilcox deleted the map_server_tests branch August 30, 2018 22:28
bpwilcox pushed a commit that referenced this pull request Sep 20, 2018
* adding map_server tests & starting port of costmap_2d package

* adding integration test node

* + integration tests

* ++ integration test

* create branch for tests
bpwilcox pushed a commit that referenced this pull request Sep 20, 2018
* adding map_server tests & starting port of costmap_2d package

* adding integration test node

* + integration tests

* ++ integration test

* create branch for tests
bpwilcox pushed a commit that referenced this pull request Sep 24, 2018
* Adding ported map_server, env_model, and env_model_msgs

* Delete gridmap.h

This file is unused / hasn't been ported to ROS2 yet

* Map server tests (#1)

* adding map_server tests & starting port of costmap_2d package

* adding integration test node

* + integration tests

* create branch for tests

* added license info, addressed formatting and white spaces, implemented tests etc.

* restructure of map_server

* additional directory refactor

* clean up package

* all test passing (cpplint, uncrustify, unit, integration, etc.)

* added TODO for publisher

* dropping yaml-cpp backwards support, changed service/topic name

* addressing feedback: changed all fprint to RCLCPP_ERROR, split main and server class, fixed yaml-cpp path, throwing exceptions up from yaml parser, cleaned cmakelist, added gtest assert throws, fixes naming conventions, added namespace

* changed class/file names, replaced fprints, took out redundant createMap, adhered to project cmakelist style and build_test, deleted unused packages

* changed name integration to component test, passes unit, component, lint tests, uncrustify, etc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant