Skip to content

Commit

Permalink
Fix hackernews after change to AssetGroup behavior (#8269)
Browse files Browse the repository at this point in the history
* with_resources top level export

* Fix where we pull assets from
  • Loading branch information
dpeng817 committed Jun 8, 2022
1 parent 44c3075 commit e8f256e
Showing 1 changed file with 9 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,18 +2,24 @@

import pytest
from dagster_pyspark import pyspark_resource
from hacker_news_assets.core import core_assets_prod
from hacker_news_assets.core import assets
from hacker_news_assets.resources.hn_resource import hn_snapshot_client
from hacker_news_assets.resources.parquet_io_manager import local_partitioned_parquet_io_manager

from dagster import AssetGroup, ResourceDefinition, fs_io_manager, mem_io_manager
from dagster import (
AssetGroup,
ResourceDefinition,
assets_from_package_module,
fs_io_manager,
mem_io_manager,
)


@pytest.mark.skip()
def test_download():
with tempfile.TemporaryDirectory() as temp_dir:
test_job = AssetGroup(
core_assets_prod.assets,
assets_from_package_module(assets),
resource_defs={
"io_manager": fs_io_manager,
"partition_start": ResourceDefinition.string_resource(),
Expand Down

0 comments on commit e8f256e

Please sign in to comment.