Skip to content
Snippets Groups Projects
Commit 20df7b01 authored by Eric Franz's avatar Eric Franz
Browse files

add refactor plan for better manifest solution

parent f3ba399a
No related branches found
Tags v0.10.1
No related merge requests found
......@@ -48,6 +48,16 @@ documentation: |
assert manifest.exist?
end
# TODO: add tests so alt yaml is used only by
# AppWithUrl so we can customize the location of the
# manifest based on the router
# then the implementation for AppWithUrl#manifest
# would be something like:
#
# @manifest ||= Manifest.load(router.manifest_path)
#
# First: refactor tests or code so its easier to make mock routers for testing
# happy path: provide access to manifest for disabled apps
# cfs.yml, cfs/manifest.yml, and cfs no rx: read cfs.yml
def test_reads_parent_yml_if_no_rx_on_appdir
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment