add tests for versioned

Author: isabelizimm

pins/boards.py

@@ -66,10 +66,6 @@ def __init__(
         self.board = str(board)
         self.fs = fs
         self.meta_factory = meta_factory
-
-        # if versioned is False:
-        #     raise NotImplementedError()
-
         self.versioned = versioned
         self.allow_pickle_read = allow_pickle_read
 
@@ -355,7 +351,9 @@ def pin_write(
         """
 
         if versioned is False:
-            raise NotImplementedError()
+            raise NotImplementedError(
+                "Can only write pins with boards that support versioning."
+            )
 
         if type == "file":
             raise NotImplementedError(

pins/constructors.py

@@ -370,7 +370,7 @@ def board_url(path: str, pin_paths: dict, cache=DEFAULT, allow_pickle_read=None)
     --------
 
     ```python
-    github_raw = "https://raw.githubusercontent.com/machow/pins-python/main/pins/tests/pins-compat"
+    github_raw = "https://raw.githubusercontent.com/rstudio/pins-python/main/pins/tests/pins-compat"
     pin_paths = {
       "df_csv": "df_csv/20220214T163720Z-9bfad/",
       "df_arrow": "df_arrow/20220214T163720Z-ad0c1/",

pins/tests/test_boards.py

@@ -600,9 +600,8 @@ def test_board_rsc_pin_read_public(df, board_short):
 
 
 def test_board_manual_http_file_download():
-    # TODO: change when repo is moved to RStudio
 
-    path = "https://raw.githubusercontent.com/machow/pins-python"
+    path = "https://raw.githubusercontent.com/rstudio/pins-python"
     license_path = "main/LICENSE"
 
     # use a simple cache, which automatically creates a temporary directory
@@ -630,7 +629,7 @@ def test_board_manual_pin_read():
     # see https://github.com/fsspec/filesystem_spec/issues/389
     fs = fsspec.filesystem("http", block_size=0)
     board = BoardManual(
-        "https://raw.githubusercontent.com/machow/pins-python/main/pins/tests/pins-compat",
+        "https://raw.githubusercontent.com/rstudio/pins-python/main/pins/tests/pins-compat",
         fs,
         pin_paths={
             "df_csv": "df_csv/20220214T163718Z-eceac/",

pins/tests/test_constructors.py

@@ -59,7 +59,7 @@ def construct_from_board(board):
 # copied from test_compat
 @pytest.mark.skip_on_github
 def test_constructor_board_url_data(tmp_cache, http_example_board_path, df_csv):
-    board = c.board_urls(
+    board = c.board_url(
         http_example_board_path,
         # could derive from example version path
         pin_paths={"df_csv": "df_csv/20220214T163720Z-9bfad/"},
@@ -77,14 +77,21 @@ def test_constructor_board_url_cache(tmp_cache, http_example_board_path, df_csv)
     # TODO: downloading a pin does not put files in the same directory, since
     # in this case we are hashing on the full url.
 
-    board = c.board_urls(
+    board = c.board_url(
         http_example_board_path,
         # could derive from example version path
         pin_paths={"df_csv": "df_csv/20220214T163718Z-eceac/"},
     )
 
     board.pin_read("df_csv")
 
+    # cannot read or view pin versions
+
+    with pytest.raises(NotImplementedError):
+        board.pin_write(df_csv)
+    with pytest.raises(NotImplementedError):
+        board.pin_versions("df_csv")
+
     # check cache ----
     http_dirs = list(tmp_cache.glob("http_*"))
 
@@ -107,7 +114,7 @@ def test_constructor_board_url_file(tmp_cache, http_example_board_path):
     # TODO: downloading a pin does not put files in the same directory, since
     # in this case we are hashing on the full url.
 
-    board = c.board_urls(
+    board = c.board_url(
         http_example_board_path,
         # could derive from example version path
         pin_paths={"df_csv": "df_csv/20220214T163718Z-eceac/df_csv.csv"},