{"push_id":2996102738,"size":1,"distinct_size":1,"ref":"refs/heads/cl/bigtest-infrastructure","head":"3d0a8fcc737a2b1fe9e3760433855d65c99ad685","before":"8117dbad2c0a872287fbdb8eb3b61f7ad07d5d25","commits":[{"sha":"3d0a8fcc737a2b1fe9e3760433855d65c99ad685","author":{"name":"Charles Lowell","email":"c83455bdc41666585a8fff1b5a28e5b7a66b0aed@frontside.io"},"message":"UI inventory BigTest harness.\n\nPurpose\n====\n\nhttps://issues.folio.org/browse/UIIN-351\n\nWe want to add both unit test coverage to the inventory application,\nand also add the ability to start the application in mirage mode so\nthat development can be done without having to actually be connected\nto any kind of server environment.\n\nApproach\n====\n\nThis adds BigTest and Mirage to the inventory app using the emerging\nstandard bigtest setup of models/serializers/factories going into the\ntests/bigtest directory.\n\nSince the endpoints were being setup against an existing API, we\nactually tested them explicitly in the\ntests/bigtest/tests/mirage-test.js directory. This let me gradually\nstandup the endpoints one at a time, verify that I could fetch from\nboth the index and the individual entity and that the chain of models,\nserializers and endpoints was working at each point.\n\nCoverage\n====\n\n```\n=============================== Coverage summary ===============================\nStatements   : 20.96% ( 215/1026 )\nBranches     : 19.47% ( 163/837 )\nFunctions    : 10.61% ( 38/358 )\nLines        : 22.13% ( 212/958 )\n================================================================================\n```\n\nStrategy\n====\n\nThere is a lot of work to be done in creating the initial stub that\nlets you achieve the very basic functionality. After that, we can\nproceed incrementally to implement features of the harness in order to\nfacillitate more comprehensive tests.\n\nAs such, the strategy was to stand up just enough of the endpoints to\nbe able to both view the application and write a basic test that\nloaded a list of inventory over the network, made sure that it\nrendered properly, and that clicking on a item in the list loaded the\ncorresponding inventory record.\n\nThat's why in this PR only the _read only_ endpoints for most entities\nhave been implemented. At such time as we want to add test coverage\nfor adding inventory or holdings, we can also make the changes to the\nharness to make that possible.\n\nBy the same token, the pagination and CQL filtering have not be\nimplemented yet with the idea being that when we add tests for\nfiltering and pagination, we can also enhance the harness.","distinct":true,"url":"https://api.github.com/repos/folio-org/ui-inventory/commits/3d0a8fcc737a2b1fe9e3760433855d65c99ad685"}]}
{"actor":{"display_login":"cowboyd"}}