@shopify/flash-list 2.0.0-alpha.21 → 2.0.0-alpha.22

package/ios/Sources/AutoLayoutView.swift

@@ -1,294 +0,0 @@
-import Foundation
-import UIKit
-
-
-/// Container for all RecyclerListView children. This will automatically remove all gaps and overlaps for GridLayouts with flexible spans.
-/// Note: This cannot work for masonry layouts i.e, pinterest like layout
-@objc public class AutoLayoutView: UIView {
-    #if RCT_NEW_ARCH_ENABLED
-    @objc public var onBlankAreaEventHandler: ((CGFloat, CGFloat) -> Void)?
-    #endif
-    
-    @objc(onBlankAreaEvent)
-    var onBlankAreaEvent: RCTDirectEventBlock?
-
-    @objc public func setHorizontal(_ horizontal: Bool) {
-        self.horizontal = horizontal
-    }
-
-    @objc public func setScrollOffset(_ scrollOffset: Int) {
-        self.scrollOffset = CGFloat(scrollOffset)
-    }
-
-    @objc public func setWindowSize(_ windowSize: Int) {
-        self.windowSize = CGFloat(windowSize)
-    }
-
-    @objc public func setRenderAheadOffset(_ renderAheadOffset: Int) {
-        self.renderAheadOffset = CGFloat(renderAheadOffset)
-    }
-
-    @objc public func setEnableInstrumentation(_ enableInstrumentation: Bool) {
-        self.enableInstrumentation = enableInstrumentation
-    }
-
-    @objc public func setDisableAutoLayout(_ disableAutoLayout: Bool) {
-        self.disableAutoLayout = disableAutoLayout
-    }
-
-    private var horizontal = false
-    private var scrollOffset: CGFloat = 0
-    private var windowSize: CGFloat = 0
-    private var renderAheadOffset: CGFloat = 0
-    private var enableInstrumentation = false
-    private var disableAutoLayout = false
-
-    /// Tracks where the last pixel is drawn in the overall
-    private var lastMaxBoundOverall: CGFloat = 0
-    /// Tracks where the last pixel is drawn in the visible window
-    private var lastMaxBound: CGFloat = 0
-    /// Tracks where first pixel is drawn in the visible window
-    private var lastMinBound: CGFloat = 0
-
-    override public func layoutSubviews() {
-        fixLayout()
-        super.layoutSubviews()
-
-        guard enableInstrumentation, let scrollView = getScrollView() else { return }
-
-        let scrollContainerSize = horizontal ? scrollView.frame.width : scrollView.frame.height
-        let currentScrollOffset = horizontal ? scrollView.contentOffset.x : scrollView.contentOffset.y
-        let startOffset = horizontal ? frame.minX : frame.minY
-        let endOffset = horizontal ? frame.maxX : frame.maxY
-        let distanceFromWindowStart = max(startOffset - currentScrollOffset, 0)
-        let distanceFromWindowEnd = max(currentScrollOffset + scrollContainerSize - endOffset, 0)
-
-        let (blankOffsetStart, blankOffsetEnd) = computeBlankFromGivenOffset(
-            currentScrollOffset - startOffset,
-            filledBoundMin: lastMinBound,
-            filledBoundMax: lastMaxBound,
-            renderAheadOffset: renderAheadOffset,
-            windowSize: windowSize,
-            distanceFromWindowStart: distanceFromWindowStart,
-            distanceFromWindowEnd: distanceFromWindowEnd
-        )
-        
-        #if RCT_NEW_ARCH_ENABLED
-        onBlankAreaEventHandler?(blankOffsetStart, blankOffsetEnd)
-        #else
-        onBlankAreaEvent?(
-            [
-                "offsetStart": blankOffsetStart,
-                "offsetEnd": blankOffsetEnd,
-            ]
-        )
-        #endif
-    }
-
-    func getScrollView() -> UIScrollView? {
-        return sequence(first: self, next: { $0.superview }).first(where: { $0 is UIScrollView }) as? UIScrollView
-    }
-
-    /// Sorts views by index and then invokes clearGaps which does the correction.
-    /// Performance: Sort is needed. Given relatively low number of views in RecyclerListView render tree this should be a non issue.
-    private func fixLayout() {
-        guard
-            subviews.count > 1,
-            // Fixing layout during animation can interfere with it.
-            layer.animationKeys()?.isEmpty ?? true,
-            !disableAutoLayout
-        else { return }
-        let cellContainers = subviews
-            .compactMap { subview -> CellContainerComponentView? in
-                if let cellContainer = subview as? CellContainerComponentView {
-                    return cellContainer
-                } else {
-                    assertionFailure("CellRendererComponent outer view should always be CellContainer. Learn more here: https://shopify.github.io/flash-list/docs/usage#cellrenderercomponent.")
-                    return nil
-                }
-            }
-            .sorted(by: { $0.index < $1.index })
-        clearGaps(for: cellContainers)
-        fixFooter()
-    }
-
-    /// Checks for overlaps or gaps between adjacent items and then applies a correction.
-    /// Performance: RecyclerListView renders very small number of views and this is not going to trigger multiple layouts on the iOS side.
-    private func clearGaps(for cellContainers: [CellContainerComponentView]) {
-        var maxBound: CGFloat = 0
-        var minBound: CGFloat = CGFloat(Int.max)
-        var maxBoundNextCell: CGFloat = 0
-        let correctedScrollOffset = scrollOffset - (horizontal ? frame.minX : frame.minY)
-        lastMaxBoundOverall = 0
-        cellContainers.indices.dropLast().forEach { index in
-            let cellContainer = cellContainers[index]
-            let cellTop = cellContainer.frame.minY
-            let cellBottom = cellContainer.frame.maxY
-            let cellLeft = cellContainer.frame.minX
-            let cellRight = cellContainer.frame.maxX
-
-            let nextCell = cellContainers[index + 1]
-            let nextCellTop = nextCell.frame.minY
-            let nextCellLeft = nextCell.frame.minX
-
-			// Only apply correction if the next cell is consecutive.
-			let isNextCellConsecutive = nextCell.index == cellContainer.index + 1
-			
-			let isCellVisible = isWithinBounds(
-				cellContainer,
-				scrollOffset: correctedScrollOffset,
-				renderAheadOffset: renderAheadOffset,
-				windowSize: windowSize,
-				isHorizontal: horizontal
-			)
-			let isNextCellVisible = isWithinBounds(
-				nextCell,
-				scrollOffset: correctedScrollOffset,
-				renderAheadOffset: renderAheadOffset,
-				windowSize: windowSize,
-				isHorizontal: horizontal
-			)
-
-            guard
-                isCellVisible || isNextCellVisible
-            else {
-                updateLastMaxBoundOverall(currentCell: cellContainer, nextCell: nextCell)
-                return
-            }
-           
-
-            if horizontal {
-                maxBound = max(maxBound, cellRight)
-                minBound = min(minBound, cellLeft)
-                maxBoundNextCell = maxBound
-				if isNextCellConsecutive {
-					if cellTop < nextCellTop {
-						if cellBottom != nextCellTop {
-							nextCell.frame.origin.y = cellBottom
-						}
-						if cellLeft != nextCellLeft {
-							nextCell.frame.origin.x = cellLeft
-						}
-					} else {
-						nextCell.frame.origin.x = maxBound
-					}
-				}
-                if isNextCellVisible {
-                    maxBoundNextCell = max(maxBound, nextCell.frame.maxX)
-                }
-            } else {
-                maxBound = max(maxBound, cellBottom)
-                minBound = min(minBound, cellTop)
-                maxBoundNextCell = maxBound
-				if isNextCellConsecutive {
-					if cellLeft < nextCellLeft {
-						if cellRight != nextCellLeft {
-							nextCell.frame.origin.x = cellRight
-						}
-						if cellTop != nextCellTop {
-							nextCell.frame.origin.y = cellTop
-						}
-					} else {
-						nextCell.frame.origin.y = maxBound
-					}
-				}
-                if isNextCellVisible {
-                    maxBoundNextCell = max(maxBound, nextCell.frame.maxY)
-                }
-            }
-            updateLastMaxBoundOverall(currentCell: cellContainer, nextCell: nextCell)
-        }
-
-        lastMaxBound = maxBoundNextCell
-        lastMinBound = minBound
-    }
-
-    private func updateLastMaxBoundOverall(currentCell: CellContainerComponentView, nextCell: CellContainerComponentView) {
-        lastMaxBoundOverall = max(lastMaxBoundOverall, horizontal ? currentCell.frame.maxX : currentCell.frame.maxY, horizontal ? nextCell.frame.maxX : nextCell.frame.maxY)
-    }
-
-    func computeBlankFromGivenOffset(
-        _ actualScrollOffset: CGFloat,
-        filledBoundMin: CGFloat,
-        filledBoundMax: CGFloat,
-        renderAheadOffset: CGFloat,
-        windowSize: CGFloat,
-        distanceFromWindowStart: CGFloat,
-        distanceFromWindowEnd: CGFloat
-    ) -> (
-        offsetStart: CGFloat,
-        offsetEnd: CGFloat
-    ) {
-        let blankOffsetStart = filledBoundMin - actualScrollOffset - distanceFromWindowStart
-
-        let blankOffsetEnd = actualScrollOffset + windowSize - renderAheadOffset - filledBoundMax - distanceFromWindowEnd
-
-        return (blankOffsetStart, blankOffsetEnd)
-    }
-
-    /// It's important to avoid correcting views outside the render window. An item that isn't being recycled might still remain in the view tree. If views outside get considered then gaps between unused items will cause algorithm to fail.
-    func isWithinBounds(
-        _ cellContainer: CellContainerComponentView,
-        scrollOffset: CGFloat,
-        renderAheadOffset: CGFloat,
-        windowSize: CGFloat,
-        isHorizontal: Bool
-    ) -> Bool {
-        let boundsStart = scrollOffset - renderAheadOffset
-        let boundsEnd = scrollOffset + windowSize
-        let cellFrame = cellContainer.frame
-
-        if isHorizontal {
-            return (cellFrame.minX >= boundsStart || cellFrame.maxX >= boundsStart) && (cellFrame.minX <= boundsEnd || cellFrame.maxX <= boundsEnd)
-        } else {
-            return (cellFrame.minY >= boundsStart || cellFrame.maxY >= boundsStart) && (cellFrame.minY <= boundsEnd || cellFrame.maxY <= boundsEnd)
-        }
-    }
-
-    /// Fixes footer position along with rest of the items
-    private func fixFooter() {
-        guard !disableAutoLayout, let parentScrollView = getScrollView() else {
-            return
-        }
-
-        let isAutoLayoutEndVisible = horizontal ? frame.maxX <= parentScrollView.frame.width : frame.maxY <= parentScrollView.frame.height
-        guard isAutoLayoutEndVisible, let footer = footer() else {
-            return
-        }
-
-        let diff = footerDiff()
-        guard diff != 0 else { return }
-
-        if horizontal {
-            footer.frame.origin.x += diff
-            frame.size.width += diff
-            superview?.frame.size.width += diff
-        } else {
-            footer.frame.origin.y += diff
-            frame.size.height += diff
-            superview?.frame.size.height += diff
-        }
-    }
-
-    private func footerDiff() -> CGFloat {
-        if subviews.count == 0 {
-            lastMaxBoundOverall = 0
-        } else if subviews.count == 1 {
-            let firstChild = subviews[0]
-            lastMaxBoundOverall = horizontal ? firstChild.frame.maxX : firstChild.frame.maxY
-        }
-        let autoLayoutEnd = horizontal ? frame.width : frame.height
-        return lastMaxBoundOverall - autoLayoutEnd
-    }
-
-    private func footer() -> UIView? {
-        // On the new arch, AutoLayoutView is wrapped with AutoLayoutViewComponentView, so we need to go up one more level
-        #if RCT_NEW_ARCH_ENABLED
-        let parentSubviews = superview?.superview?.subviews
-        #else
-        let parentSubviews = superview?.subviews
-        #endif
-
-        return parentSubviews?.first(where:{($0 as? CellContainerComponentView)?.index == -1})
-    }
-}

package/ios/Sources/AutoLayoutViewComponentView.h

@@ -1,16 +0,0 @@
-#ifndef AutoLayoutViewComponentView_h
-#define AutoLayoutViewComponentView_h
-
-#ifdef RCT_NEW_ARCH_ENABLED
-
-#import <UIKit/UIKit.h>
-#import <React/RCTViewComponentView.h>
-
-@interface AutoLayoutViewComponentView : RCTViewComponentView
-
-@end
-
-
-#endif /* RCT_NEW_ARCH_ENABLED */
-
-#endif /* AutoLayoutViewComponentView_h */

package/ios/Sources/AutoLayoutViewComponentView.mm

@@ -1,90 +0,0 @@
-#ifdef RCT_NEW_ARCH_ENABLED
-#import "AutoLayoutViewComponentView.h"
-#import <React/RCTConversions.h>
-#import <React/RCTViewManager.h>
-
-#import <react/renderer/components/rnflashlist/ComponentDescriptors.h>
-#import <react/renderer/components/rnflashlist/EventEmitters.h>
-#import <react/renderer/components/rnflashlist/Props.h>
-#import <react/renderer/components/rnflashlist/RCTComponentViewHelpers.h>
-
-#import "RCTFabricComponentsPlugins.h"
-
-#if __has_include(<RNFlashList/RNFlashList-Swift.h>)
-#import <RNFlashList/RNFlashList-Swift.h>
-#else
-#import "RNFlashList-Swift.h"
-#endif
-
-using namespace facebook::react;
-
-@interface AutoLayoutViewComponentView () <RCTAutoLayoutViewViewProtocol>
-@end
-
-@implementation AutoLayoutViewComponentView
-{
-    AutoLayoutView *_autoLayoutView;
-}
-
-- (instancetype)initWithFrame:(CGRect)frame
-{
-  if (self = [super initWithFrame:frame]) {
-    static const auto defaultProps = std::make_shared<const AutoLayoutViewProps>();
-    _props = defaultProps;
-    _autoLayoutView = [[AutoLayoutView alloc] initWithFrame:self.bounds];
-    self.contentView = _autoLayoutView;
-
-    __weak AutoLayoutViewComponentView* weakSelf = self;
-    _autoLayoutView.onBlankAreaEventHandler = ^(CGFloat start, CGFloat end) {
-      AutoLayoutViewComponentView *strongSelf = weakSelf;
-      if (strongSelf != nullptr && strongSelf->_eventEmitter != nullptr) {
-        std::dynamic_pointer_cast<const facebook::react::AutoLayoutViewEventEmitter>(strongSelf->_eventEmitter)
-          ->onBlankAreaEvent(facebook::react::AutoLayoutViewEventEmitter::OnBlankAreaEvent{
-            .offsetStart = (int) floor(start),
-            .offsetEnd = (int) floor(end),
-        });
-      }
-    };
-  }
-
-  return self;
-}
-
-- (void)mountChildComponentView:(UIView<RCTComponentViewProtocol> *)childComponentView index:(NSInteger)index
-{
-  [_autoLayoutView mountChildComponentView:childComponentView index:index];
-}
-
-- (void)unmountChildComponentView:(UIView<RCTComponentViewProtocol> *)childComponentView index:(NSInteger)index
-{
-  [_autoLayoutView unmountChildComponentView:childComponentView index:index];
-}
-
-#pragma mark - RCTComponentViewProtocol
-
-+ (ComponentDescriptorProvider)componentDescriptorProvider
-{
-  return concreteComponentDescriptorProvider<AutoLayoutViewComponentDescriptor>();
-}
-
-- (void)updateProps:(const Props::Shared &)props oldProps:(const Props::Shared &)oldProps
-{
-    const auto &newProps = *std::static_pointer_cast<const AutoLayoutViewProps>(props);
-
-    [_autoLayoutView setHorizontal:newProps.horizontal];
-    [_autoLayoutView setScrollOffset:newProps.scrollOffset];
-    [_autoLayoutView setWindowSize:newProps.windowSize];
-    [_autoLayoutView setRenderAheadOffset:newProps.renderAheadOffset];
-    [_autoLayoutView setEnableInstrumentation:newProps.enableInstrumentation];
-    [_autoLayoutView setDisableAutoLayout:newProps.disableAutoLayout];
-
-    [super updateProps:props oldProps:oldProps];
-}
-@end
-
-Class<RCTComponentViewProtocol> AutoLayoutViewCls(void)
-{
-  return AutoLayoutViewComponentView.class;
-}
-
-#endif /* RCT_NEW_ARCH_ENABLED */

package/ios/Sources/AutoLayoutViewManager.mm

@@ -1,14 +0,0 @@
-#import <Foundation/Foundation.h>
-#import <React/RCTViewManager.h>
-
-@interface RCT_EXTERN_MODULE(AutoLayoutViewManager, RCTViewManager)
-
-RCT_EXPORT_VIEW_PROPERTY(horizontal, BOOL)
-RCT_EXPORT_VIEW_PROPERTY(scrollOffset, NSInteger)
-RCT_EXPORT_VIEW_PROPERTY(windowSize, NSInteger)
-RCT_EXPORT_VIEW_PROPERTY(renderAheadOffset, NSInteger)
-RCT_EXPORT_VIEW_PROPERTY(enableInstrumentation, BOOL)
-RCT_EXPORT_VIEW_PROPERTY(disableAutoLayout, BOOL)
-RCT_EXPORT_VIEW_PROPERTY(onBlankAreaEvent, RCTDirectEventBlock)
-
-@end

package/ios/Sources/AutoLayoutViewManager.swift

@@ -1,12 +0,0 @@
-import Foundation
-
-@objc(AutoLayoutViewManager)
-class AutoLayoutViewManager: RCTViewManager {
-    override func view() -> UIView! {
-        return AutoLayoutView()
-    }
-    
-    override static func requiresMainQueueSetup() -> Bool {
-        return true
-    }
-}

package/ios/Sources/CellContainerComponentView.h

@@ -1,18 +0,0 @@
-#ifndef CellContainer_h
-#define CellContainer_h
-
-#import <UIKit/UIKit.h>
-
-#ifdef RCT_NEW_ARCH_ENABLED
-#import <React/RCTViewComponentView.h>
-
-@interface CellContainerComponentView : RCTViewComponentView
-#else
-@interface CellContainerComponentView : UIView
-#endif // RCT_NEW_ARCH_ENABLED
-
-@property int64_t index;
-
-@end
-
-#endif /* CellContainer_h */

package/ios/Sources/CellContainerComponentView.mm

@@ -1,62 +0,0 @@
-#import "CellContainerComponentView.h"
-
-#ifdef RCT_NEW_ARCH_ENABLED
-#import <React/RCTConversions.h>
-
-#import <react/renderer/components/rnflashlist/ComponentDescriptors.h>
-#import <react/renderer/components/rnflashlist/EventEmitters.h>
-#import <react/renderer/components/rnflashlist/Props.h>
-#import <react/renderer/components/rnflashlist/RCTComponentViewHelpers.h>
-
-#import "RCTFabricComponentsPlugins.h"
-
-#if __has_include(<RNFlashList/RNFlashList-Swift.h>)
-#import <RNFlashList/RNFlashList-Swift.h>
-#else
-#import "RNFlashList-Swift.h"
-#endif
-
-using namespace facebook::react;
-
-@interface CellContainerComponentView () <RCTCellContainerViewProtocol>
-@end
-
-@implementation CellContainerComponentView
-
-- (instancetype)initWithFrame:(CGRect)frame
-{
-    if (self = [super initWithFrame:frame]) {
-        static const auto defaultProps = std::make_shared<const CellContainerProps>();
-        _props = defaultProps;
-
-        self.userInteractionEnabled = true;
-    }
-
-    return self;
-}
-
-#pragma mark - RCTComponentViewProtocol
-
-+ (ComponentDescriptorProvider)componentDescriptorProvider
-{
-    return concreteComponentDescriptorProvider<CellContainerComponentDescriptor>();
-}
-
-- (void)updateProps:(const Props::Shared &)props oldProps:(const Props::Shared &)oldProps
-{
-    const auto &newProps = *std::static_pointer_cast<const CellContainerProps>(props);
-
-    self.index = newProps.index;
-
-    [super updateProps:props oldProps:oldProps];
-}
-@end
-
-Class<RCTComponentViewProtocol> CellContainerCls(void)
-{
-    return CellContainerComponentView.class;
-}
-#else
-@implementation CellContainerComponentView
-@end
-#endif /* RCT_NEW_ARCH_ENABLED */

package/ios/Sources/CellContainerManager.mm

@@ -1,8 +0,0 @@
-#import <Foundation/Foundation.h>
-#import <React/RCTViewManager.h>
-
-@interface RCT_EXTERN_MODULE(CellContainerManager, RCTViewManager)
-
-RCT_EXPORT_VIEW_PROPERTY(index, NSInteger)
-
-@end

package/ios/Sources/CellContainerManager.swift

@@ -1,12 +0,0 @@
-import Foundation
-
-@objc(CellContainerManager)
-class CellContainerManager: RCTViewManager {  
-    override func view() -> UIView! {
-        return CellContainerComponentView()
-    }
-    
-    override static func requiresMainQueueSetup() -> Bool {
-        return true
-    }
-}

package/ios/Sources/FlatListPro-Bridging-Header.h

@@ -1,11 +0,0 @@
-#ifndef FlatListPro_Bridging_Header_h
-#define FlatListPro_Bridging_Header_h
-
-#import <React/RCTBridgeModule.h>
-#import <React/RCTEventEmitter.h>
-#import <React/RCTViewManager.h>
-#import <React/RCTComponent.h>
-
-#import "CellContainerComponentView.h"
-
-#endif /* FlatListPro_Bridging_Header_h */

package/ios/Tests/AutoLayoutViewTests.swift

@@ -1,113 +0,0 @@
-import XCTest
-@testable import RNFlashList
-
-class AutoLayoutViewTests: XCTestCase {
-    var autoLayoutView: AutoLayoutView!
-    var testCell: CellContainer!
-    let cellHeight: CGFloat = 50
-    let windowSize: CGFloat = 500
-
-    override func setUp() {
-        super.setUp()
-        autoLayoutView = AutoLayoutView()
-        testCell = CellContainer()
-    }
-
-    func testIsWithinBoundsVerticalListScrollToBottom() {
-        testCell.frame = CGRect(x: 0, y: 0, width: cellHeight, height: cellHeight)
-        // Top left corner of the screen
-        XCTAssertTrue(autoLayoutView.isWithinBounds(testCell,
-                                                    scrollOffset: 0,
-                                                    renderAheadOffset: 0,
-                                                    windowSize: windowSize,
-                                                    isHorizontal: false))
-
-        // Partly visible
-        XCTAssertTrue(autoLayoutView.isWithinBounds(testCell,
-                                                    scrollOffset: 25,
-                                                    renderAheadOffset: 0,
-                                                    windowSize: windowSize,
-                                                    isHorizontal: false))
-
-        // Above visible
-        XCTAssertFalse(autoLayoutView.isWithinBounds(testCell,
-                                                     scrollOffset: 100,
-                                                     renderAheadOffset: 0,
-                                                     windowSize: windowSize,
-                                                     isHorizontal: false))
-    }
-
-    func testIsWithinBoundsVerticalListScrollToTop() {
-        testCell.frame = CGRect(x: 0, y: windowSize - cellHeight, width: cellHeight, height: cellHeight)
-        // Bottom left corner of the screen
-        XCTAssertTrue(autoLayoutView.isWithinBounds(testCell,
-                                                    scrollOffset: 0,
-                                                    renderAheadOffset: 0,
-                                                    windowSize: windowSize,
-                                                    isHorizontal: false))
-
-        // Partly visible
-        XCTAssertTrue(autoLayoutView.isWithinBounds(testCell,
-                                                    scrollOffset: -25,
-                                                    renderAheadOffset: 0,
-                                                    windowSize: windowSize,
-                                                    isHorizontal: false))
-
-        // Below visible
-        XCTAssertFalse(autoLayoutView.isWithinBounds(testCell,
-                                                     scrollOffset: -100,
-                                                     renderAheadOffset: 0,
-                                                     windowSize: windowSize,
-                                                     isHorizontal: false))
-    }
-
-    func testIsWithinBoundsHorizontalListScrollToLeft() {
-        testCell.frame = CGRect(x: 0, y: 0, width: cellHeight, height: cellHeight)
-        // Top left corner of the screen
-        XCTAssertTrue(autoLayoutView.isWithinBounds(testCell,
-                                                    scrollOffset: 0,
-                                                    renderAheadOffset: 0,
-                                                    windowSize: windowSize,
-                                                    isHorizontal: true))
-
-        // Partly visible
-        XCTAssertTrue(autoLayoutView.isWithinBounds(testCell,
-                                                    scrollOffset: 25,
-                                                    renderAheadOffset: 0,
-                                                    windowSize: windowSize,
-                                                    isHorizontal: true))
-
-        // Hidden
-        XCTAssertFalse(autoLayoutView.isWithinBounds(testCell,
-                                                     scrollOffset: 100,
-                                                     renderAheadOffset: 0,
-                                                     windowSize: windowSize,
-                                                     isHorizontal: true))
-    }
-
-    func testIsWithinBoundsHorizontalListScrollToRight() {
-        testCell.frame = CGRect(x: windowSize - cellHeight, y: 0, width: cellHeight, height: cellHeight)
-        // Bottom left corner of the screen
-        XCTAssertTrue(autoLayoutView.isWithinBounds(testCell,
-                                                    scrollOffset: 0,
-                                                    renderAheadOffset: 0,
-                                                    windowSize: windowSize,
-                                                    isHorizontal: true))
-
-        // Partly visible
-        XCTAssertTrue(autoLayoutView.isWithinBounds(testCell,
-                                                    scrollOffset: -25,
-                                                    renderAheadOffset: 0,
-                                                    windowSize: windowSize,
-                                                    isHorizontal: true))
-
-        testCell.frame = CGRect(x: windowSize + cellHeight, y: 0, width: cellHeight, height: cellHeight)
-        // Hidden
-        XCTAssertFalse(autoLayoutView.isWithinBounds(testCell,
-                                                     scrollOffset: 0,
-                                                     renderAheadOffset: 0,
-                                                     windowSize: windowSize,
-                                                     isHorizontal: true))
-    }
-
-}