Commit graph

27 commits

Author SHA1 Message Date
Jeff Bailey
b9ede32f5b Issue #299 Unit tests for IGListCollectionView
Summary:
Add 2 new unit test files for IGListCollectionView.  One is for storyboards and the other for when creating the IGListCollectionView in code.

- [X] All tests pass. Demo project builds and runs.
- [X] I added tests, an experiment, or detailed why my change isn't tested.
- [X] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/.github/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/303

Differential Revision: D4304615

Pulled By: rnystrom

fbshipit-source-id: c6edbcd493fe4ee087ce3d0aee80db484207859d
2016-12-08 20:14:01 -08:00
Ryan Nystrom
0395ca31e9 Add unit tests to reloadData updater
Summary:
Adding a few more missing unit tests.

Closes #185
Closes https://github.com/Instagram/IGListKit/pull/289

Differential Revision: D4284610

Pulled By: rnystrom

fbshipit-source-id: a592fb9c12a3995a62b608f44cdcc693a0f79bec
2016-12-06 05:59:08 -08:00
Ryan Nystrom
10bdfb23f9 Add more unit tests to stack section controller
Summary:
Beefing up our test coverage. Made an improvement to supplementary view behavior along the way. Will update `CHANGELOG.md` once travis finishes w/ link to PR #.

- [x] All tests pass. Demo project builds and runs.
- [x] I added tests, an experiment, or detailed why my change isn't tested.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/.github/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/286

Differential Revision: D4281961

Pulled By: jessesquires

fbshipit-source-id: 32b5877bd72250b9a99e600ceffc64d686fa5651
2016-12-05 17:44:02 -08:00
Bofei Zhu
4231235620 Unit tests for -scrollToObject horizontal end position
Summary:
Fixes #238

Try to finish what I started 😂
Closes https://github.com/Instagram/IGListKit/pull/247

Differential Revision: D4220011

Pulled By: rnystrom

fbshipit-source-id: 9f4e1bd4f112f69f601c83d6604dc18bc27f0c68
2016-11-22 06:44:01 -08:00
Jesse Squires
acd3ea217e Cleanup layout
Summary:
Replaces #243

---------------------

- Clean up GridLayout + docs
- Remove GridLayout scroll direction, not used/supported?
- Fix some file target membership issues
- Add GridLayout to umbrella header
- "Sort by name" on xcode project directories
- crtl-I auto format in xcode
- re-gen docs
Closes https://github.com/Instagram/IGListKit/pull/246

Differential Revision: D4219028

Pulled By: jessesquires

fbshipit-source-id: 02d6d284c47dd4301549d9094727054156d596aa
2016-11-21 23:28:58 -08:00
Ryan Nystrom
316fbe2b8b Remove NSObject+IGListDiffable category
Summary: Removing the category entirely (recreated for unit tests and examples). Went through and updated all of Instagram so we don't rely on the category and found a few models that needed cleaning up.

Reviewed By: jessesquires

Differential Revision: D4206552

fbshipit-source-id: ee8147e8872ea19f6c8bd8072df61eed5bffb6f3
2016-11-21 16:59:00 -08:00
Jesse Squires
cb45684e4d Regen docs for 2.0 release. Close #181
Summary:
Closes #181
Closes https://github.com/Instagram/IGListKit/pull/207

Differential Revision: D4216383

Pulled By: jessesquires

fbshipit-source-id: 664c6d0f1cf82647cf8e2231cf35348b08351c9d
2016-11-21 15:44:01 -08:00
Bofei Zhu
1d3e58b9fd Create a grid layout for IGList
Summary:
Rebase from #140:

Create a flow layout for IGList that does not newline sections. Closes #3

- [x] Minimum inter-item spacing
- [x] Minimum line spacing
- [x] Constant item size with constant layout time
- [x] Update layout on insert/delete/move
- [x] Unit Test
Closes https://github.com/Instagram/IGListKit/pull/225

Differential Revision: D4211469

Pulled By: rnystrom

fbshipit-source-id: f4710dbf195701098ac50f94b6b2aa8c801b2a83
2016-11-21 13:29:04 -08:00
Ryan Nystrom
f05ae5b145 Fix TV tests
Summary:
Saw in [travis jobs](https://travis-ci.org/Instagram/IGListKit/jobs/177139465) that TV tests were failing. Fixed some file target memberships. Not sure why failing TV tests wouldn't fail Travis tho.

Maybe fix #227

If we don't get coverage reports I'll amend removing `cd $TAVIS_BUILD_DIR` and see what happens.

- [x] All tests pass. Demo project builds and runs.
- [x] I added tests, an experiment, or detailed why my change isn't tested.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/228

Differential Revision: D4210696

Pulled By: rnystrom

fbshipit-source-id: 695c1211353790163e1dd11176b2dc801d83e3f7
2016-11-19 16:13:55 -08:00
Ryan Nystrom
7a4ebc94bc Add stricter build flags
Summary:
Gets the `xcodeproj` in parity with our internal build settings.

Fixes #211

- [x] All tests pass. Demo project builds and runs.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/224

Differential Revision: D4210413

Pulled By: rnystrom

fbshipit-source-id: c820faf97ef04bc7cf7c369446d0e17829b6339b
2016-11-19 11:58:58 -08:00
Ryan Nystrom
2660b14084 Test adapter proxy object
Summary:
Fixes #186

- [x] All tests pass. Demo project builds and runs.
- [x] I added tests, an experiment, or detailed why my change isn't tested.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/218

Differential Revision: D4201040

Pulled By: rnystrom

fbshipit-source-id: ef94ea7d77af60abe19dc611f5390b9a8bf25f6a
2016-11-18 11:13:59 -08:00
Ryan Nystrom
24174dcbf6 Unit test moves in containers
Summary:
Unit testing moves in sets so their `hash` and `isEqual` functions provide uniqueness.

Fixes #187, #188

- [x] All tests pass. Demo project builds and runs.
- [x] I added tests, an experiment, or detailed why my change isn't tested.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/217

Differential Revision: D4201042

Pulled By: jessesquires

fbshipit-source-id: 9c4803ca11e04bb86a7745d6ebc63d173e8ac814
2016-11-17 18:59:00 -08:00
Bofei Zhu
379d68e5c1 Commit 1a92c15 breaks pbxproj
Summary:
Update project.pbxproj
Closes https://github.com/Instagram/IGListKit/pull/174

Differential Revision: D4147048

Pulled By: rnystrom

fbshipit-source-id: 03713030f5604839af3c64f64f3a700e08b3fa94
2016-11-08 06:29:05 -08:00
Ryan Nystrom
576ab4d1ff Remove NSIndexPath pretty print
Summary:
Saw this pop up in coverage reports. We don't log or assert `NSIndexPath` at all so this doesn't really seem necessary.

- [x] All tests pass. Demo project builds and runs.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/165

Differential Revision: D4138474

Pulled By: rnystrom

fbshipit-source-id: ad8375e47d1584f594d3df29f07a1d5d1a53f3c5
2016-11-06 12:44:10 -08:00
Ryan Nystrom
f5339113df Support supplementaryViews created from nibs (continued)
Summary:
Continuing the work on #90. I don't believe I can push directly to that PR since the origin is `master` of a repo I don't have access to.

https://help.github.com/articles/checking-out-pull-requests-locally/

I went ahead and added another supplementary view test copying the old one we had.

cc jessesquires in case there's something else I can do here. I believe this will still give rawlinxx credit?
Closes https://github.com/Instagram/IGListKit/pull/162

Differential Revision: D4137364

Pulled By: rnystrom

fbshipit-source-id: d8418ac5728fd6d9570fa1d1568f4343f5c4112b
2016-11-05 16:29:06 -07:00
Bofei Zhu
158d04ebe0 Support supplementaryViews created from storyboard
Summary:
Miss this in the previous PR #92. Will add unit test soon.
Closes https://github.com/Instagram/IGListKit/pull/123

Differential Revision: D4101594

Pulled By: rnystrom

fbshipit-source-id: 820030358532b0878f6d9e9092834266c9260a38
2016-11-01 10:59:21 -07:00
Jesse Squires
e4a47193c1 Add tvOS support. Close #118
Summary:
Closes #118.

- Add tvOS framework + test targets
- Add shared scheme `IGListKit-tvOS`
- Update `Podfile` and `.podspec`
- Update `.travis.yml`
- Update `CHANGELOG`
- Update `README`
- Run `pod install` and check-in updates

- [x] All tests pass. Demo project builds and runs.
- [x] I added tests, an experiment, or detailed why my change isn't tested.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/137

Differential Revision: D4101583

Pulled By: rnystrom

fbshipit-source-id: f2736f16ad9f61ad5348c5e93571f3b752deec98
2016-10-29 14:44:11 -07:00
Ayush Saraswat
ebd8801f90 Rename *ItemController unit tests to *SectionController
Summary:
Rename *ItemController unit tests to *SectionController, addressing issue #126

- [x] All tests pass. Demo project builds and runs.
- [x] I added tests, an experiment, or detailed why my change isn't tested.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/129

Differential Revision: D4095496

Pulled By: rnystrom

fbshipit-source-id: 277be38ed2789f14199cee9d711a838552921f10
2016-10-28 07:44:16 -07:00
Ryan Nystrom
b5aa5e3900 Replace diff result API for batch updates
Summary:
Replacing the move+update API with a batch-updates-safe API on the diff results object. This makes using the diff results w/out the rest of IGListKit infra much easier when working with `UITableView` or `UICollectionView`.

- Added unit tests
- Removed outdated unit tests

Reviewed By: dshahidehpour

Differential Revision: D4065798

fbshipit-source-id: 30da8a7b483d56d5acc497da9320dc07a6d0b7ad
2016-10-26 11:59:13 -07:00
Bofei Zhu
8855b5156a Add storyboard support #39
Summary:
This PR is for #39

- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/92

Differential Revision: D4066721

Pulled By: rnystrom

fbshipit-source-id: 4fca278ae9a54aa46626932d47d4a4c001b437d6
2016-10-25 15:29:13 -07:00
Ryan Nystrom
d78c0b4e13 Add Coveralls support and badge
Summary:
Adding Coverall support and badge in README. Waiting on tests. Fixes #37

- [x] All tests pass. Demo project builds and runs.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/110

Differential Revision: D4066685

Pulled By: rnystrom

fbshipit-source-id: a9f148fe87e9d25c60afca88c02410898db9b10a
2016-10-23 17:29:11 -07:00
Ryan Nystrom
8a96fc9c64 Extension only API flag
Summary:
Adds `APPLICATION_EXTENSION_API_ONLY = YES` to Xcode project.
Closes https://github.com/Instagram/IGListKit/pull/79

Differential Revision: D4028543

Pulled By: rnystrom

fbshipit-source-id: 203afacb5485dd4dc5897991d7981cf80f6da993
2016-10-16 19:44:06 -07:00
Ryan Nystrom
d4f64c3e90 Swap test nib to test bundle
Summary:
IGTestNibCell.xib was in the wrong bundle after imported internally (moved it to `Assets` dir)

- [x] All tests pass. Demo project builds and runs.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/78

Reviewed By: ryanolsonk

Differential Revision: D4028444

Pulled By: rnystrom

fbshipit-source-id: cb7ed0f8024ca9fd116827f3935f075c1d9193be
2016-10-16 16:44:02 -07:00
Sven Bacia
ce4b851da3 Support cells created from nibs
Summary:
I started working on adding support for dequeuing cells created from nibs (issue #1). Additionally I extended `IGListSingleSectionController` so that it can be used with nibs too. I don't know if you had this also in mind.

- [x]  I'm currently thinking about the best way to test these changes.
- [x] I was not able to update the documentation (issue #55).

- [x] All tests pass. Demo project builds and runs.
- [x] I added tests, an experiment, or detailed why my change isn't tested.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/CONTRIBUTING.md)
Closes https://github.com/Instagram/IGListKit/pull/56

Reviewed By: dshahidehpour

Differential Revision: D4023746

Pulled By: rnystrom

fbshipit-source-id: 6a8b4cfb4dba38ea6e9870a9a4506288ee155cfe
2016-10-14 18:59:18 -07:00
Ryan Nystrom
d8a48943c3 Pod file updates
Summary: Pod files were skipped after API renames.

Reviewed By: jessesquires

Differential Revision: D4003106

fbshipit-source-id: d3b5336a8e98880549d4414d679cbeff94169a01
2016-10-11 16:25:32 -04:00
Ryan Nystrom
1483e8e99b Fix intern syncing with xcodeproj 2016-10-11 15:24:47 -04:00
FBShipIt
cdc796746a Initial commit
fbshipit-source-id: c38b897b9e2b87c782e0e01b0145d9189dc7f9d8
2016-10-11 10:12:17 -04:00