forked from Open-IoT-Service-Platform/platform-launcher
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Develop2 #8
Open
wagmarcel
wants to merge
68
commits into
master
Choose a base branch
from
develop2
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Also added some file to .gitignore.
This is not required any more due to changes in postgres and frontend containers.
This is not necessary anymore because the content is built-in the containner. When the volume is mounted anyway, the app won't run, because node-modules directory where the modules are installed in the previous build step will be overwritten.
The volume is no longer required as the source is moved into the container.
… The test DB is created when postgres container is started the first time. This avoids the error when frontend tries to recreate the DB. Postgres get now a special variable POSTGRES_DB_REGULAR which is always the name of the production DB. Testdb name is now always 'test'
…etrieved in the right order, or not received at all. Solved by adding function to check whether new emails arrive before trying to retrieve it. Moved whole mails.js to Promises. Added function to consume/delete emails before processing next ones. Added function to download and delete all emails in box. Changed the Rule engine tests: email arrival is not checked. Added test for bulk checking whether all emails from rule-engine tests arrived
* Removed docker-compose-quick, a single file is used for pulling and manually building. * Refactored Makefile heavily, and added documentation * Moved to a new version of docker-compose in platform setup * Made the matching changes to setup-environment.example.sh
…started due to Kafka not being ready.
-Observation test now with timing, e.g. send one observation per second -Number of actuations is checked before resolving tests to also count unwanted actuations -Added helper to create TB-rules
…y exists on the device
ifconfig is deprecated and replaced by ip
After Jaeger agent container has been added to k8s (for monitoring) there are multiple containers in the frontend, and it is necessary to specify that the test command should run in 'dashboard' Signed-off-by: Ali Rasim Kocal <[email protected]>
* Fixed receive tests: comparison of number of returned points with expected number of points * Comparison of float values is testing abs difference larger than MIN_NUMBER rather than equality * Added configuration for opentsdb and moved the testing timestamps up to 100000000000 as base * Implemented cleanup of subtest * Added 2s between sending and receiving * setup-environment.example.sh: Added description how to configure tsdbName in OISP_BACKEND_CONFIG
…or opentsdb container
…bmsission implementation in frontend * Sending partially correct data, i.e. with wrong component IDs. Expected behaviour is that correct data is submitted and PartialDataProcessed(404) error is returned * Correcting 0 prefix in test data which is not tolerated with nodejs v8 * Correcting some indentation issues * Added test for Admin sending data and User trying to send data and get rejected * Added test to send data to device not belonging to account (Should get NotAuthorized (401)) * Added tests on checking whether device can send data to wrong device (should get NotAuthorized 401
e6d5582 introduces a second user, which must be created when testing with k8s as well Signed-off-by: Ali Rasim Kocal <[email protected]>
Signed-off-by: Ali Rasim Kocal <[email protected]>
…le, disable jaeger by default Signed-off-by: Oguzcan Kirmemis <[email protected]>
* Added oisp-mqtt-gw submodule, * Adapted to OISP config structure and added to setup-environment.sh * Added broker and gateway to main docker-compose * Added cafile, certfile, etc to broker config.
Signed-off-by: Marcel Wagner <[email protected]>
…kend Signed-off-by: Marcel Wagner <[email protected]>
This should close Open-IoT-Service-Platform#120 Signed-off-by: Ali Rasim Kocal <[email protected]>
Signed-off-by: Ali Rasim Kocal <[email protected]>
Signed-off-by: Ali Rasim Kocal <[email protected]>
…(images, videos, ...) - Created new classes for Data, Rules and Components - Removed the all hard coded test values - Added an image component to test the binary encoding feature - Updated oisp-sdk-js hash to make sure sdk supports cbor Signed-off-by: Marcel Wagner <[email protected]>
Signed-off-by: Marcel Wagner <[email protected]>
Signed-off-by: Marcel Wagner <[email protected]>
* Added config file with sign-off checking * First test for ssh triggering with circleci * Updated ci script to avoid interactive host checking Signed-off-by: Marcel Wagner <[email protected]>
…kefile realizes that compose failed Signed-off-by: Marcel Wagner <[email protected]>
Added scnd kafka listener for localhost Signed-off-by: Marcel Wagner <[email protected]>
Signed-off-by: Marcel Wagner <[email protected]>
Signed-off-by: Marcel Wagner <[email protected]>
Signed-off-by: Marcel Wagner <[email protected]>
Signed-off-by: Marcel Wagner <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.