Skip to content

Feature/Mongo API#32

Open
ASHuenchuleo wants to merge 4 commits intomainfrom
feature/mongo-api
Open

Feature/Mongo API#32
ASHuenchuleo wants to merge 4 commits intomainfrom
feature/mongo-api

Conversation

@ASHuenchuleo
Copy link
Contributor

Description

Added feature to allow direct requests to the new mongo API

Related Issue or Feature

Issue #26

Motivation and Context

As part of the migration to Mongo, an API has been developed to proxy direct queries to the database. This client facilitates the connection process just like witht he PSQL API.

How Has This Been Tested?

Tests were added to the existing unit tests.

@ASHuenchuleo ASHuenchuleo linked an issue Jan 26, 2023 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Jan 26, 2023

Codecov Report

Merging #32 (93aa3cf) into main (6a09655) will increase coverage by 0.79%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   87.02%   87.82%   +0.79%     
==========================================
  Files           7        7              
  Lines         370      386      +16     
==========================================
+ Hits          322      339      +17     
+ Misses         48       47       -1     
Impacted Files Coverage Δ
alerce/core.py 100.00% <100.00%> (ø)
alerce/direct.py 100.00% <100.00%> (ø)
alerce/utils.py 97.95% <100.00%> (+1.02%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7c55758...93aa3cf. Read the comment docs.

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.

Update client to use API with mongo functionalities

2 participants