Skip to content

Commit decbd84

Browse files
committed
fixed duplicate requests in NewsFeedMeta
1 parent 4a12572 commit decbd84

File tree

4 files changed

+17
-2
lines changed

4 files changed

+17
-2
lines changed

CHANGELOG

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
1+
v1.4.14
2+
- fixed duplicate requests in NewsFeedMeta
3+
14
v1.4.13
25
- added SystemInfoSearch model
36

VERSION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
1.4.13
1+
1.4.14

lib/spark_api/models/news_feed_meta.rb

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,11 @@ def core_fields
3131
private
3232

3333
def data
34-
@data ||= connection.get(self.path).first
34+
if attributes.empty?
35+
@data ||= connection.get(self.path).first
36+
else
37+
attributes
38+
end
3539
end
3640

3741
end

spec/unit/spark_api/models/news_feed_meta_spec.rb

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,14 @@
1010
stub_api_get("/newsfeeds/meta", "newsfeeds/meta.json")
1111
end
1212

13+
describe 'fetching data' do
14+
it "only makes one api request" do
15+
news_feed_meta = NewsFeedMeta.get.first
16+
news_feed_meta.minimum_core_fields
17+
expect_api_request(:get, "/newsfeeds/meta").to have_been_made.once
18+
end
19+
end
20+
1321
describe 'minimum_core_fields' do
1422
it 'returns the minimum number of required fields' do
1523
expect(news_feed_meta.minimum_core_fields).to eq 3

0 commit comments

Comments
 (0)