Providers名前空間の切り出し #276

AnaTofuZ · 2018-03-05T02:34:12Z

背景

#270 で events を作るに辺りstatisticsとは別途イベントにAPIにアクセスし直近のイベント情報を入手する必要が出てきた。
そこで、statistics/providers を再利用したいが、名前空間的に切り出したほうが良いと考えられる為、切り出しを行う

行ったこと

lib/statistics/providers 以下のコードを lib/providers にmvした。
テストコードで利用しているモジュールパスをそれに伴い修正した
providers を利用しているコードのパスを対応させた

`/events`を作りたい為、雛形hamlファイルとコントローラー及び適切に`/events` が表示できるかのテストコードを追加した。

statiscs/providersにはDojoのイベントAPIにアクセスする為の実装がある. 直近のイベント情報を取得する際も、APIアクセスが必要となる為この資産を使いたい。 > statistics 系の資源を使いつつ、直近情報を入手する実装がやっぱり良さそう > ネームスペースが気持ち悪いと思うので、Statisticsの外に出してしまうと良いかもしれないですねぇ。そこで名前空間をstatiscs/providersからprovidersに変更した

先程のcommitでnamespaceの変更を行った為、Statisticsの名前空間を削除した

…o.jp into add_events_pages

`providers` の名前空間を`statistics` から外した為、require_relativeのパスを修正した

Statistics::ProvidersからProvides::に名前空間を修正した為使用しているrakeファイルの名前空間を修正した

テストコードが`Statistics::Providers` を参照していた為、`Statistics` 名前空間を外した

nalabjp · 2018-03-05T02:48:39Z

ありがとうございます！
/eventsでも使うならネームスペース変えたいなぁと思ってたところでした👍

個人的にはStatisticsとは別のネームスペースを持つか、Providersから別のネームスペースに変えたいと思ってたのですがいかがでしょうか？
lib配下のモジュールでProvidersだけだとちょっと意味が広いなぁという印象を持っています。

AnaTofuZ · 2018-03-05T02:52:51Z

@nalabjp ありがとうございます!　個人的には API/Providers あたりなのかな…?と思いましたが、あまり良いネームスペースの設計が思いつかないので、ご教授お願いしたいです。

また名前空間の変更のPRなのに以前の /events の作業時のcommitが残ってしまっていたので一旦rmしたいと思います

今回のPRでは`/events` の変更は行わない為削除した

nalabjp · 2018-03-05T03:12:35Z

Api::Providersでも良いと思いますよ😌

僕が付けそうなのは
テーブル名の一部としても使っているevent_serviceを使ってEventService::Providersにするとかですかね。
あとはStatistics::ProvidersからApiClientにしてしまうとかでしょうか😉

yasulab · 2018-03-05T03:15:05Z

EventService::Providers

こっちの方がしっくりきますね! 😸 今回はイベント管理サービス以外の目的でAPIを叩かないですし、また、テーブル名と同じなので「(/stats ページなどで表示されているデータから) そのファイルにどんなコードが入っているか?」も類推しやすくて良いですね! 📑👀💭

AnaTofuZ · 2018-03-05T03:20:58Z

@nalabjp @yasulab ありがとうございます! EventService::Providers しっくり来ますね!
EventService::Providers でネームスペースを揃えてみます(๑•̀ㅂ•́)و✧

…b module 名前空間を切り分けるために eventservice/providers.rbにproviders.rbを変更し大本のeventservice.rbを作成した

providersの名前空間をeventservice以下にする為テストコードの名前空間を切り分けた

…e::Providers

名前空間を変更した為,lib/tasks/statisticsの使用しているモジュールパスを変更した

名前空間を変更した物を修正し忘れていた為変更

AnaTofuZ · 2018-03-05T05:12:27Z

名前空間を EventService::Providers で修正しました。
その為 eventservice.rb などを定義しています

nalabjp · 2018-03-11T05:12:21Z

以下の理由より、ファイル名とディレクトリ名をs/eventservice/event_service/したいです 🙏

5.1.4@2.4.2 (main)> 'eventservice'.camelize
=> "Eventservice"
5.1.4@2.4.2 (main)> 'event_service'.camelize
=> "EventService"
5.1.4@2.4.2 (main)> 'EventService'.underscore
=> "event_service"

nalabjp

#276 (comment)
以外はLGTMです😌

AnaTofuZ · 2018-03-12T01:13:11Z

@nalabjp
なるほど理解です! (Railsの命名規則)
ファイル名とディレクトリ名を修正変更してみます

…_service/

AnaTofuZ · 2018-03-12T01:23:37Z

CIでコケていたのは

An error occurred while loading ./spec/lib/eventservice/providers/connpass_spec.rb.
Failure/Error: require 'eventservice'

とありテストコードのrequireの部分が修正していなかったからだった為修正を行う

前回のcommit時と同様に名前空間のrenameを行う coderdojo-japan@f6469ff

命名規則の修正でファイルを直接requireしている部分をeven_serviceに修正した

AnaTofuZ · 2018-03-12T02:24:38Z

名前の切り分けが終了しました!

nalabjp · 2018-03-12T03:04:30Z

ゴメンナサイ、見落としていたのですが、Statistics::Clientも一緒にEventService配下に持っていきませんか？😅

AnaTofuZ · 2018-03-12T03:05:57Z

Statistics::Client はStatisticsに特化しているのかなと思ってたのですが、たしかにコードを見るとそうでもないですね…こちらも移動してみます

AnaTofuZ · 2018-03-12T03:17:16Z

一折切り分けがこれで完了したかと思います!

nalabjp

LGTM

AnaTofuZ added 8 commits February 28, 2018 10:32

touch events.html.haml and add static_pages_controller_spec test

c02f95c

`/events`を作りたい為、雛形hamlファイルとコントローラー及び適切に`/events` が表示できるかのテストコードを追加した。

remove module Statistics namespace for lib/providers/*.rb

1a03b3c

先程のcommitでnamespaceの変更を行った為、Statisticsの名前空間を削除した

Merge branch 'add_events_pages' of ssh://github.com/AnaTofuZ/coderdoj…

cb0a915

…o.jp into add_events_pages

lib/statistics.rbでの providers の名前空間からstatisticsを外した

4b8f227

`providers` の名前空間を`statistics` から外した為、require_relativeのパスを修正した

名前空間の変更に伴うlib/tasks/statistics.rakeでのStatistics::Providersの名前修正

ec1f07b

Statistics::ProvidersからProvides::に名前空間を修正した為使用しているrakeファイルの名前空間を修正した

git mv spec/lib/statistics/providers -> spec/lib/providers/*

1e412e4

spec/lib/providers* でのテスト対象の名前空間の修正

4721898

テストコードが`Statistics::Providers` を参照していた為、`Statistics` 名前空間を外した

AnaTofuZ added 2 commits March 5, 2018 11:55

rm /events components

6219af4

今回のPRでは`/events` の変更は行わない為削除した

remove spec/controller/static_pages_controller_spec for /evens

f341803

AnaTofuZ added 8 commits March 5, 2018 12:21

git mv providers/* -> eventservice/providers/*

f609b2d

move providers.rb to eventservice/providers.rb and new eventservice.r…

b30cdbd

…b module 名前空間を切り分けるために eventservice/providers.rbにproviders.rbを変更し大本のeventservice.rbを作成した

add eventservice name space for lib/**/*.rb

fd426de

modified lib/eventservice/providers/*.rb indents

d044d00

git mv spec/lib/providers -> spec/lib/eventservice/providers

a20eac9

providersの名前空間をeventservice以下にする為テストコードの名前空間を切り分けた

modofied spec/lib/eventservice/providers use Providers -> EventServic…

0fd4ef8

…e::Providers

modofied lib/task/statistics use Providers -> EventService::Providers

4fc5347

名前空間を変更した為,lib/tasks/statisticsの使用しているモジュールパスを変更した

use Everything namespace for lib/statistics/tasks/*.rb

5e052ab

名前空間を変更した物を修正し忘れていた為変更

AnaTofuZ mentioned this pull request Mar 6, 2018

直近の CoderDojo 開催情報を表示したい #258

Closed

nalabjp approved these changes Mar 11, 2018

View reviewed changes

rename directory and lib/eventservice for git mv s/eventservice/event…

f6469ff

…_service/

AnaTofuZ added 2 commits March 12, 2018 10:24

rename spec/lib/eventservice/* -> spec/lib/event_service

a6fa303

前回のcommit時と同様に名前空間のrenameを行う coderdojo-japan@f6469ff

modofied require 'eventservice' -> 'event_service'

29a838f

命名規則の修正でファイルを直接requireしている部分をeven_serviceに修正した

AnaTofuZ added 4 commits March 12, 2018 12:07

mv statistics/client.rb -> event_service/client.rb

272a297

modofied use event_service/client

c026250

modified namespace Statistics -> EventService

56602fb

modofied use test code for Statistics::Client -> EventService::Client

0f4dcd5

nalabjp approved these changes Mar 12, 2018

View reviewed changes

AnaTofuZ merged commit e43a707 into coderdojo-japan:master Mar 12, 2018

AnaTofuZ deleted the modified_providers_namespace branch March 12, 2018 04:47

aokabin mentioned this pull request Apr 13, 2018

[WIP]直近のイベント情報を集計するtaskの追加 #297

Closed

2 tasks

Uh oh!

Providers名前空間の切り出し #276

Providers名前空間の切り出し #276

Uh oh!

Conversation

AnaTofuZ commented Mar 5, 2018

背景

行ったこと

Uh oh!

nalabjp commented Mar 5, 2018

Uh oh!

AnaTofuZ commented Mar 5, 2018

Uh oh!

nalabjp commented Mar 5, 2018

Uh oh!

yasulab commented Mar 5, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

AnaTofuZ commented Mar 5, 2018

Uh oh!

AnaTofuZ commented Mar 5, 2018

Uh oh!

nalabjp commented Mar 11, 2018

Uh oh!

nalabjp left a comment

Choose a reason for hiding this comment

Uh oh!

AnaTofuZ commented Mar 12, 2018

Uh oh!

AnaTofuZ commented Mar 12, 2018

Uh oh!

AnaTofuZ commented Mar 12, 2018

Uh oh!

nalabjp commented Mar 12, 2018

Uh oh!

AnaTofuZ commented Mar 12, 2018

Uh oh!

AnaTofuZ commented Mar 12, 2018

Uh oh!

nalabjp left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

yasulab commented Mar 5, 2018 •

edited

Loading