@Osyotr has signed the CLA from Pull Request #21

Just gtfs improvements

.github/workflows/cla.yml

@@ -0,0 +1,34 @@
+name: "CLA Assistant"
+on:
+  issue_comment:
+    types: [created]
+  pull_request_target:
+    types: [opened,closed,synchronize]
+
+jobs:
+  CLAssistant:
+    runs-on: ubuntu-latest
+    steps:
+      - name: "CLA Assistant"
+        if: (github.event.comment.body == 'recheck' || github.event.comment.body == 'I have read the CLA Document and I hereby sign the CLA') || github.event_name == 'pull_request_target'
+        # Alpha Release
+        uses: cla-assistant/github-action@v2.1.0-alpha
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          PERSONAL_ACCESS_TOKEN : ${{ secrets.PERSONAL_ACCESS_TOKEN }}
+        with:
+          path-to-signatures: 'signatures/version1/cla.json'
+          path-to-document: 'https://github.com/mapsme/just_gtfs/blob/master/MIT_CLA.md'
+          # branch should not be protected
+          branch: 'master'
+          allowlist: mesozoic-drones,tatiana-yan,bot*
+
+         #below are the optional inputs - If the optional inputs are not given, then default values will be taken
+          #remote-organization-name: enter the remote organization name where the signatures should be stored (Default is storing the signatures in the same repository)
+          #remote-repository-name:  enter the  remote repository name where the signatures should be stored (Default is storing the signatures in the same repository)
+          #create-file-commit-message: 'For example: Creating file for storing CLA Signatures'
+          #signed-commit-message: 'For example: $contributorName has signed the CLA in #$pullRequestNo'
+          #custom-notsigned-prcomment: 'pull request comment with Introductory message to ask new contributors to sign'
+          #custom-pr-sign-comment: 'The signature to be committed in order to sign the CLA'
+          #custom-allsigned-prcomment: 'pull request comment when all contributors has signed, defaults to **CLA Assistant Lite bot** All Contributors have signed the CLA.'
+

CMakeLists.txt

@@ -11,5 +11,8 @@ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Werror")
 
 enable_testing()
 
+add_library(just_gtfs INTERFACE)
+target_include_directories(just_gtfs INTERFACE ${CMAKE_CURRENT_LIST_DIR}/include)
+
 add_subdirectory(tests)
 add_subdirectory(benchmarks)

MIT_CLA.md

@@ -0,0 +1,59 @@
+## Individual Contributor License Agreement (CLA)
+
+**Thank you for submitting your contributions to this project.**
+
+By signing this CLA, you agree that the following terms apply to all of your past, present and future contributions
+to the project.
+
+### License.
+
+You hereby represent that all present, past and future contributions are governed by the
+[MIT License](https://opensource.org/licenses/MIT)
+copyright statement.
+
+This entails that to the extent possible under law, you transfer all copyright and related or neighboring rights
+of the code or documents you contribute to the project itself or its maintainers.
+Furthermore you also represent that you have the authority to perform the above waiver
+with respect to the entirety of you contributions.
+
+### Moral Rights.
+
+To the fullest extent permitted under applicable law, you hereby waive, and agree not to
+assert, all of your “moral rights” in or relating to your contributions for the benefit of the project.
+
+### Third Party Content.
+
+If your Contribution includes or is based on any source code, object code, bug fixes, configuration changes, tools,
+specifications, documentation, data, materials, feedback, information or other works of authorship that were not
+authored by you (“Third Party Content”) or if you are aware of any third party intellectual property or proprietary
+rights associated with your Contribution (“Third Party Rights”),
+then you agree to include with the submission of your Contribution full details respecting such Third Party
+Content and Third Party Rights, including, without limitation, identification of which aspects of your
+Contribution contain Third Party Content or are associated with Third Party Rights, the owner/author of the
+Third Party Content and Third Party Rights, where you obtained the Third Party Content, and any applicable
+third party license terms or restrictions respecting the Third Party Content and Third Party Rights. For greater
+certainty, the foregoing obligations respecting the identification of Third Party Content and Third Party Rights
+do not apply to any portion of a Project that is incorporated into your Contribution to that same Project.
+
+### Representations.
+
+You represent that, other than the Third Party Content and Third Party Rights identified by
+you in accordance with this Agreement, you are the sole author of your Contributions and are legally entitled
+to grant the foregoing licenses and waivers in respect of your Contributions. If your Contributions were
+created in the course of your employment with your past or present employer(s), you represent that such
+employer(s) has authorized you to make your Contributions on behalf of such employer(s) or such employer
+(s) has waived all of their right, title or interest in or to your Contributions.
+
+### Disclaimer.
+
+To the fullest extent permitted under applicable law, your Contributions are provided on an "as is"
+basis, without any warranties or conditions, express or implied, including, without limitation, any implied
+warranties or conditions of non-infringement, merchantability or fitness for a particular purpose. You are not
+required to provide support for your Contributions, except to the extent you desire to provide support.
+
+### No Obligation.
+
+You acknowledge that the maintainers of this project are under no obligation to use or incorporate your contributions
+into the project. The decision to use or incorporate your contributions into the project will be made at the
+sole discretion of the maintainers or their authorized delegates.
+

README.md

@@ -1,85 +1,74 @@
-# just_gtfs - header-only modern C++ GTFS parsing library
+# just_gtfs - header-only modern C++ library for reading and writing GTFS feeds
 
-[![GTFS parser for C++](https://github.com/mapsme/just_gtfs/blob/add-the-most-important-readers/docs/logo.jpeg)](https://github.com/mapsme/just_gtfs)
+[![GTFS reader and writer for C++](https://github.com/mapsme/just_gtfs/blob/add-the-most-important-readers/docs/logo.jpeg)](https://github.com/mapsme/just_gtfs)
 
 [![C++](https://img.shields.io/badge/c%2B%2B-17-informational.svg)](https://shields.io/)
 [![MIT license](https://img.shields.io/badge/License-MIT-blue.svg)](https://lbesson.mit-license.org/)
 ![](https://github.com/mapsme/just_gtfs/workflows/C%2FC%2B%2B%20CI/badge.svg)
+[![](https://github.com/sindresorhus/awesome/blob/main/media/mentioned-badge.svg)](https://github.com/CUTR-at-USF/awesome-transit)
 [![contributions welcome](https://img.shields.io/badge/contributions-welcome-brightgreen.svg?style=flat)](https://github.com/mapsme/just_gtfs/issues)
 
- - Header-only
- - C++17
- - Tested on GCC and Clang
- - STL-compatible containers
- - Fast reading and parsing of GTFS feeds
 
 ## Table of Contents
-- [Working with GTFS feeds](#working-with-gtfs-feeds)
-- [How to use just_library](#how-to-use-it)
+- [Description](#description)
+- [Reading and writing GTFS feeds](#reading-and-writing-gtfs-feeds)
+- [How to add library to your project](#how-to-add-library-to-your-project)
 - [Used third-party tools](#used-third-party-tools)
+- [Contributing](#contributing)
+- [Resources](#resources)
 
-## Working with GTFS feeds
-The library implements reading static transit data in GTFS - [General Transit Feed Specification](https://developers.google.com/transit/gtfs/reference).
-It provides class for working with GTFS feeds: `gtfs::Feed`.
-GTFS csv files are mapped to the corresponding C++ classes. Every GTFS entity can be accessed through `gtfs::Feed`.
+## Description
+The just_gtfs library implements reading and writing static transit data in GTFS - [General Transit Feed Specification](https://developers.google.com/transit/gtfs/reference).
 
-:pushpin: Example of providing `gtfs::Feed` the feed path, reading it and working with GTFS entities such as stops and routes:
-```c++
-Feed feed("~/data/SFMTA/");
-if (feed.read_feed() == ResultCode::OK)
-{
-	Stops stops = feed.get_stops();
-	std::cout << stops.size() << std::endl;
-
-	Route route = feed.get_route("route_id_1009");
-	if (route)
-	{
-		std::cout << route->route_long_name << std::endl;
-	}
-}
-```
-
-GTFS feed can be wholly read from directory as in the example above or you can read GTFS files separately. E.g., if you need only shapes data, you can avoid parsing all other files and just work with the shapes.
-
-:pushpin: Example of reading only `shapes.txt` from the feed and working with shapes:
-```c++
-Feed feed("~/data/SFMTA/");
-if (feed.read_shapes() == ResultCode::OK)
-{
-	Shapes all_shapes = feed.get_shapes();
-	Shape shape = feed.get_shape("9367");
-}
-```
+Its main features:
+ - Fast reading and writing of GTFS feeds
+ - Support for [extended GTFS route types](https://developers.google.com/transit/gtfs/reference/extended-route-types)
+ - Simple working with GTFS `Date` and `Time` formats
+ - Header-only
+ - Written in C++17
+ - Tested on GCC and Clang
 
 
-## Methods for reading and writing GTFS entities
-Methods of the `Feed` class for working with agencies:
+## Reading and writing GTFS feeds
+Library provides main class for working with GTFS feeds: `gtfs::Feed`. It also provides classes for each of the 17 GTFS entities: `Route`, `Stop`, `Pathway`, `Translation` and others.
+GTFS csv files are mapped to the corresponding C++ classes. Every GTFS entity can be accessed through `gtfs::Feed` corresponding getters & setters.
 
-Read agencies from the corresponding csv file.
+:pushpin: All GTFS entities are managed in the same way. So here is the example for working with `agencies`.
+
+Method of the `Feed` class for reading `agency.txt`:
 ```c++
 Result read_agencies()
 ```
 
-Get reference to `Agencies` - `std::vector` of `Agency` objects.
+Method for reading reading not only agencies but all GTFS entities. Path to the feed is specified in the `Feed` constructor:
+```c++
+Result read_feed()
+```
+
+Method for getting reference to the `Agencies` - `std::vector` of all `Agency` objects of the feed:
 ```c++
 const Agencies & get_agencies()
 ```
 
-Find agency by its id. This method returns `std::optional` so you should check if the result is `std::nullopt`.
+Method for finding agency by its id. Returns `std::optional` so you should check if the result is `std::nullopt`:
 ```c++
 std::optional<Agency> get_agency(const Id & agency_id)
 ``` 
 
-Add agency to the feed.
+Method for adding agency to the feed:
 ```c++
 void add_agency(const Agency & agency)
 ```
 
-Add agency to the feed by filling agency object fields with parsed csv values. `row` is `std::map` with csv column titles as keys ans csv row items as values. 
+Method for writing agencies to the `agency.txt` file to `gtfs_path`.
 ```c++
-Result add_agency(ParsedCsvRow const & row)
+Result write_agencies(const std::string & gtfs_path)
 ```
 
+Method for writing all GTFS entities (not only agencies, but stops, stop times, calendar etc):
+```c++
+Result write_feed(const std::string & gtfs_path)
+```
 
 :pushpin: **There are similar methods for all other GTFS entities** for getting the list of entities, finding and adding them.
 For some of them additional methods are provided. 
@@ -93,9 +82,63 @@ Or you can find stop times for the particular trip:
 StopTimes get_stop_times_for_trip(const Id & trip_id, bool sort_by_sequence = true)
 ```
 
-## How to use library
-- For including the library in your own project: just_gtfs is completely contained inside a single header and therefore it is sufficient to copy include/just_gtfs/just_gtfs.h to your include pathes. The library does not have to be explicitly build.
-- For running library tests:
+### Example of reading GTFS feed and working with its stops and routes
+:pushpin: Provide `gtfs::Feed` the feed path, read it and work with GTFS entities such as stops and routes:
+```c++
+Feed feed("~/data/SFMTA/");
+if (feed.read_feed() == ResultCode::OK)
+{
+  Stops stops = feed.get_stops();
+  std::cout << "Stops count in feed: " << stops.size() << std::endl;
+
+  for (const Stop & stop: stops)
+  {
+    std::cout << stop.stop_id << std::endl;
+  }
+
+  Route route = feed.get_route("route_id_1009");
+  if (route)
+  {
+    std::cout << route->route_long_name << std::endl;
+  }
+}
+```
+
+### Example of parsing shapes.txt and working with its contents
+GTFS feed can be wholly read from directory as in the example above or you can read GTFS files separately. E.g., if you need only shapes data, you can avoid parsing all other files and just work with the shapes.
+
+:pushpin: Read only `shapes.txt` from the feed and work with shapes:
+```c++
+Feed feed("~/data/SFMTA/");
+if (feed.read_shapes() == ResultCode::OK)
+{
+  Shapes all_shapes = feed.get_shapes();
+  Shape shape = feed.get_shape("9367");
+
+  for (const ShapePoint & point: shape)
+  {
+    std::cout << point.shape_pt_lat << " " << point.shape_pt_lon << std::endl;
+  }
+}
+```
+
+### Example of writing GTFS:
+:pushpin: If you already filled the `feed` object with data that suits you, you can write it to the corresponding path:
+```c++
+Feed feed;
+
+// Fill feed with agencies, stops, routes and other required data:
+
+feed.add_trip(some_trip);
+feed.add_attribution(attr);
+
+feed.write_feed("~/data/custom_feed/");
+```
+
+## How to add library to your project
+- For including just_gtfs to your own project **as a submodule:** use branch "for-usage-as-submodule" which consists of a single header.
+- Another way of including just_gtfs to your project: just_gtfs is completely contained inside a single header and therefore it is sufficient to copy include/just_gtfs/just_gtfs.h to your **include paths.** The library does not have to be explicitly build.
+- For building library and **running tests:**
 Clone just_gtfs with `git clone --recursive` or run `git submodule update --init --recursive --remote` after cloning.
 In the just_gtfs project directory build the project and run unit tests: 
 ```
@@ -103,8 +146,15 @@ cmake .
 make
 ctest --output-on-failure --verbose
 ```
-The library makes use of the C++17 features and therefore you have to use appropriate compiler version.
-- For including as a submodule: use branch "for-usage-as-submodule" which consists of a single header.
+The library makes use of the C++17 features and therefore you have to use the appropriate compiler version.
 
 ## Used third-party tools
 - [**doctest**](https://github.com/onqtam/doctest) for unit testing.
+
+## Contributing
+Please open a [Github issue](https://github.com/mapsme/just_gtfs/issues/new) with as much of the information as you're able to specify, or create a [pull request](https://github.com/mapsme/just_gtfs/pulls) according to our [guidelines](https://github.com/mapsme/just_gtfs/blob/master/docs/CPP_STYLE.md).
+
+## Resources
+[GTFS reference in Google GitHub repository](https://github.com/google/transit/blob/master/gtfs/spec/en/reference.md)
+
+[GTFS reference on Google Transit API](https://developers.google.com/transit/gtfs/reference?csw=1)

signatures/version1/cla.json

@@ -0,0 +1,20 @@
+{
+  "signedContributors": [
+    {
+      "name": "nilsnolde",
+      "id": 25637358,
+      "comment_id": 1441674396,
+      "created_at": "2023-02-23T12:23:33Z",
+      "repoId": 250751634,
+      "pullRequestNo": 18
+    },
+    {
+      "name": "Osyotr",
+      "id": 8740768,
+      "comment_id": 2089322842,
+      "created_at": "2024-05-02T00:13:58Z",
+      "repoId": 250751634,
+      "pullRequestNo": 21
+    }
+  ]
+}

tests/data/output_feed/agency.txt

@@ -0,0 +1,3 @@
+agency_id,agency_name,agency_url,agency_timezone,agency_lang,agency_phone,agency_fare_url,agency_email
+0Id_b^3 Company,"Big Big ""Bus Company""",,,,,b3c.no,b3c@gtfs.com
+kwf,"""killer whale ferries""",,Asia/Tokyo,en,842,f@mail.com,

tests/data/sample_feed/fare_attributes.txt

@@ -1,3 +1,4 @@
 fare_id,price,currency_type,payment_method,transfers,transfer_duration
 p,1.25,USD,0,0,
-a,5.25,USD,0,0,
+a,5.25,USD,1,1,
+x,20,USD,0,,60

tests/unit_tests.cpp

@@ -23,6 +23,14 @@ TEST_CASE("Time in HH:MM:SS format")
   CHECK_EQ(stop_time.get_total_seconds(), 39 * 60 * 60 + 45 * 60 + 30);
 }
 
+TEST_CASE("Time in HHH:MM:SS format")
+{
+  Time stop_time("103:05:21");
+  CHECK_EQ(stop_time.get_hh_mm_ss(), std::make_tuple(103, 5, 21));
+  CHECK_EQ(stop_time.get_raw_time(), "103:05:21");
+  CHECK_EQ(stop_time.get_total_seconds(), 103 * 60 * 60 + 5 * 60 + 21);
+}
+
 TEST_CASE("Time from integers 1")
 {
   Time stop_time(14, 30, 0);
@@ -44,6 +52,7 @@ TEST_CASE("Invalid time format")
   CHECK_THROWS_AS(Time("12/10/00"), const InvalidFieldFormat &);
   CHECK_THROWS_AS(Time("12:100:00"), const InvalidFieldFormat &);
   CHECK_THROWS_AS(Time("12:10:100"), const InvalidFieldFormat &);
+  CHECK_THROWS_AS(Time("12:10/10"), const InvalidFieldFormat &);
 }
 
 TEST_CASE("Time not provided")
@@ -151,9 +160,69 @@ TEST_CASE("Quotation marks")
   CHECK_EQ(res[4], "44.29124");
   CHECK_EQ(res[5], "1");
 }
+
+TEST_CASE("Not wrapped quotation marks")
+{
+  const auto res = CsvParser::split_record(R"(Contains "quotes", commas and text)");
+  REQUIRE_EQ(res.size(), 2);
+  CHECK_EQ(res[0], R"(Contains "quotes")");
+  CHECK_EQ(res[1], "commas and text");
+}
+
+TEST_CASE("Wrapped quotation marks")
+{
+  const auto res = CsvParser::split_record(R"("Contains ""quotes"", commas and text")");
+  REQUIRE_EQ(res.size(), 1);
+  CHECK_EQ(res[0], R"(Contains "quotes", commas and text)");
+}
+
+TEST_CASE("Double wrapped quotation marks")
+{
+  const auto res = CsvParser::split_record(R"(""Double quoted text"")");
+  REQUIRE_EQ(res.size(), 1);
+}
+
+TEST_CASE("Read quoted empty values")
+{
+  const auto res = CsvParser::split_record(",\"\"");
+  REQUIRE_EQ(res.size(), 2);
+  CHECK_EQ(res[0], "");
+  CHECK_EQ(res[1], "");
+}
+TEST_CASE("Read quoted quote")
+{
+  const auto res = CsvParser::split_record(",\"\"\"\"");
+  REQUIRE_EQ(res.size(), 2);
+  CHECK_EQ(res[0], "");
+  CHECK_EQ(res[1], "\"");
+}
+
+TEST_CASE("Read quoted double quote")
+{
+  const auto res = CsvParser::split_record(",\"\"\"\"\"\"");
+  REQUIRE_EQ(res.size(), 2);
+  CHECK_EQ(res[0], "");
+  CHECK_EQ(res[1], "\"\"");
+}
+
+TEST_CASE("Read quoted values with quotes in begin")
+{
+  const auto res = CsvParser::split_record(",\"\"\"Name\"\" and some other\"");
+  REQUIRE_EQ(res.size(), 2);
+  CHECK_EQ(res[0], "");
+  CHECK_EQ(res[1], "\"Name\" and some other");
+}
+
+TEST_CASE("Read quoted values with quotes at end")
+{
+  const auto res = CsvParser::split_record(",\"Text and \"\"Name\"\"\"");
+  REQUIRE_EQ(res.size(), 2);
+  CHECK_EQ(res[0], "");
+  CHECK_EQ(res[1], "Text and \"Name\"");
+}
 TEST_SUITE_END();
 
-TEST_SUITE_BEGIN("Read");
+TEST_SUITE_BEGIN("Read & write");
 // Credits:
 // https://developers.google.com/transit/gtfs/examples/gtfs-feed
 TEST_CASE("Empty container before parsing")
@@ -241,7 +310,7 @@ TEST_CASE("Read GTFS feed")
   CHECK_EQ(feed.get_attributions().size(), 1);
   CHECK_EQ(feed.get_calendar().size(), 2);
   CHECK_EQ(feed.get_calendar_dates().size(), 1);
-  CHECK_EQ(feed.get_fare_attributes().size(), 2);
+  CHECK_EQ(feed.get_fare_attributes().size(), 3);
   CHECK_EQ(feed.get_fare_rules().size(), 4);
   CHECK(!feed.get_feed_info().feed_publisher_name.empty());
   CHECK_EQ(feed.get_levels().size(), 3);
@@ -264,6 +333,11 @@ TEST_CASE("Agency")
 
   const auto agency = feed.get_agency("DTA");
   CHECK(agency);
+
+  REQUIRE_EQ(feed.write_agencies("data/output_feed"), ResultCode::OK);
+  Feed feed_copy("data/output_feed");
+  REQUIRE_EQ(feed_copy.read_agencies(), ResultCode::OK);
+  CHECK_EQ(agencies, feed_copy.get_agencies());
 }
 
 TEST_CASE("Routes")
@@ -321,7 +395,7 @@ TEST_CASE("Stops")
   CHECK_EQ(stops[0].stop_id, "FUR_CREEK_RES");
   CHECK(stops[0].stop_desc.empty());
   CHECK_EQ(stops[0].stop_name, "Furnace Creek Resort (Demo)");
-  CHECK_EQ(stops[0].location_type, StopLocationType::GenericNode);
+  CHECK_EQ(stops[0].location_type, StopLocationType::StopOrPlatform);
   CHECK(stops[0].zone_id.empty());
 
   auto const & stop = feed.get_stop("FUR_CREEK_RES");
@@ -420,7 +494,7 @@ TEST_CASE("Fare attributes")
   REQUIRE_EQ(feed.read_fare_attributes(), ResultCode::OK);
 
   const auto & attributes = feed.get_fare_attributes();
-  REQUIRE_EQ(attributes.size(), 2);
+  REQUIRE_EQ(attributes.size(), 3);
   CHECK_EQ(attributes[0].fare_id, "p");
   CHECK_EQ(attributes[0].price, 1.25);
   CHECK_EQ(attributes[0].currency_type, "USD");
@@ -428,9 +502,28 @@ TEST_CASE("Fare attributes")
   CHECK_EQ(attributes[0].transfers, FareTransfers::No);
   CHECK_EQ(attributes[0].transfer_duration, 0);
 
+  CHECK_EQ(attributes[1].fare_id, "a");
+  CHECK_EQ(attributes[1].price, 5.25);
+  CHECK_EQ(attributes[1].currency_type, "USD");
+  CHECK_EQ(attributes[1].payment_method, FarePayment::BeforeBoarding);
+  CHECK_EQ(attributes[1].transfers, FareTransfers::Once);
+  CHECK_EQ(attributes[1].transfer_duration, 0);
+
+  CHECK_EQ(attributes[2].fare_id, "x");
+  CHECK_EQ(attributes[2].price, 20);
+  CHECK_EQ(attributes[2].currency_type, "USD");
+  CHECK_EQ(attributes[2].payment_method, FarePayment::OnBoard);
+  CHECK_EQ(attributes[2].transfers, FareTransfers::Unlimited);
+  CHECK_EQ(attributes[2].transfer_duration, 60);
+
   const auto & attributes_for_id = feed.get_fare_attributes("a");
   REQUIRE_EQ(attributes_for_id.size(), 1);
   CHECK_EQ(attributes_for_id[0].price, 5.25);
+
+  REQUIRE_EQ(feed.write_fare_attributes("data/output_feed"), ResultCode::OK);
+  Feed feed_copy("data/output_feed");
+  REQUIRE_EQ(feed_copy.read_fare_attributes(), ResultCode::OK);
+  CHECK_EQ(attributes, feed_copy.get_fare_attributes());
 }
 
 TEST_CASE("Fare rules")
@@ -534,3 +627,34 @@ TEST_CASE("Feed info")
 }
 
 TEST_SUITE_END();
+
+TEST_SUITE_BEGIN("Simple pipelines");
+
+TEST_CASE("Agencies create & save")
+{
+  Feed feed_for_writing;
+
+  Agency agency1;
+  agency1.agency_id = "0Id_b^3 Company";
+  agency1.agency_name = R"(Big Big "Bus Company")";
+  agency1.agency_email = "b3c@gtfs.com";
+  agency1.agency_fare_url = "b3c.no";
+
+  Agency agency2;
+  agency2.agency_id = "kwf";
+  agency2.agency_name = R"("killer whale ferries")";
+  agency2.agency_lang = "en";
+  agency2.agency_phone = "842";
+  agency2.agency_timezone = "Asia/Tokyo";
+  agency2.agency_fare_url = "f@mail.com";
+
+  feed_for_writing.add_agency(agency1);
+  feed_for_writing.add_agency(agency2);
+
+  REQUIRE_EQ(feed_for_writing.write_agencies("data/output_feed"), ResultCode::OK);
+  Feed feed_for_testing("data/output_feed");
+
+  REQUIRE_EQ(feed_for_testing.read_agencies(), ResultCode::OK);
+  CHECK_EQ(feed_for_writing.get_agencies(), feed_for_testing.get_agencies());
+}
+TEST_SUITE_END();