Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

feat: memory infrastructure #46

Merged
merged 4 commits into from
Sep 15, 2021
Merged

Conversation

mimoham24
Copy link
Contributor

Overview

memory infrastructure

What I've done

create memory infrastructure
create memory test cases

What I haven't done

How I tested

unit-testing

Which point I want you to review particularly

Memo

@mimoham24 mimoham24 changed the title Tag/memory infrastructure feat: memory infrastructure Sep 12, 2021
@codecov
Copy link

codecov bot commented Sep 12, 2021

Codecov Report

❗ No coverage uploaded for pull request base (tag-system@9d2fcec). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head cee8624 differs from pull request most recent head 3c4bb63. Consider uploading reports for the commit 3c4bb63 to get more accurate results
Impacted file tree graph

@@              Coverage Diff              @@
##             tag-system      #46   +/-   ##
=============================================
  Coverage              ?   16.56%           
=============================================
  Files                 ?      266           
  Lines                 ?    42792           
  Branches              ?        0           
=============================================
  Hits                  ?     7090           
  Misses                ?    34697           
  Partials              ?     1005           

Copy link
Member

@HideBa HideBa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@rot1024 rot1024 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If possible, write unit tests for each method.

… into tag/memory-infrastructure

# Conflicts:
#	internal/infrastructure/memory/tag.go
@mimoham24
Copy link
Contributor Author

If possible, write unit tests for each method.

done! but I merged anyway

@mimoham24 mimoham24 merged commit e61cb38 into tag-system Sep 15, 2021
@mimoham24 mimoham24 deleted the tag/memory-infrastructure branch September 15, 2021 12:50
mimoham24 added a commit that referenced this pull request Oct 25, 2021
* feat: tag system domain models (#39)

* feat: tag system domain models

* refactor: * add tag interface * tag -> group and tag->item conversation

* testing: generate test cases for the tagID

* resolve notes

* fix unit tests errors

* add NewId test code
fix NewId func

* add more test cases
refactor some parts

* feat: tag system data-layer (mongo) (#44)

* feat: tag system data-layer (mongo)

* remove len > 0 check

* goimport

* Update pkg/tag/group_builder.go

Co-authored-by: rot1024 <aayhrot@gmail.com>

* Update pkg/tag/item_builder.go

Co-authored-by: rot1024 <aayhrot@gmail.com>

* rename itemFrom and groupFrom funcs

Co-authored-by: rot1024 <aayhrot@gmail.com>

* feat: create tag group and tag item (#45)

* tag item and group schema

* feat: creat tags (GQL schema)

* tag items and tag groups resolvers

* datalayer (dummy memory) and usecases

* receive list by reference

* check if nil for list

* resolve notes

* generate new models

* feat: memory infrastructure (#46)

* refactor: implement memory infrastructure

* test: implement memory infrastructure test cases

* test: fix FindByScene test case

* feat: attach/detach tag from layer (#50)

* tag item and group schema

* feat: creat tags (GQL schema)

* tag items and tag groups resolvers

* datalayer (dummy memory) and usecases

* receive list by reference

* check if nil for list

* feat: introduce tags to layers

* feat: attach/detach tags from layers

* fix imports

* refactor: resolve notes

* test: test units for tags

* refactor: resolve notes

* feat: attach/detach tag item from group (#52)

* refactor: transform group tags list to reference

* feat: attach/detach tags

* refactor: use params as use-case input

* test: mongodoc testing (#61)

* test: mongodoc testing

* resolve notes

* feat: remove tag (#58)

* feat: remove a tag (init)

* feat: remove tag

* feat: remove tag usecase and infra

* resolve notes

* feat: tag system queries (#54)

* feat: tag system queries

* resolve notes

* feat: update tag (#49)

* tag item and group schema

* feat: creat tags (GQL schema)

* tag items and tag groups resolvers

* datalayer (dummy memory) and usecases

* receive list by reference

* check if nil for list

* feat: rename tag group

* refactor: rename -> update

* resolve notes

* resolve notes

* change find by id func return type

* Merge branch 'tag-system' of https://github.com/reearth/reearth-backend into tag/update-group-label

# Conflicts:
#	internal/adapter/gql/generated.go
#	internal/adapter/gql/gqlmodel/convert_tag.go

* fix testing

* resolve notes

* resolve notes

* resolve notes

Co-authored-by: HideBa <baba.papa1120.ba@gmail.com>
Co-authored-by: rot1024 <aayhrot@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants