meteor / meteor/module-todos-react

Remove this repo and redirect to meteor/todos

Open
#3 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
32
Forks
8
PR merge metrics
No merged PRs in 30d

Description

I would love to see tome example tests... I just finished the Meteor 1.3 Tutorial and was trying to create my first test in meteor.

Following the example that `remove` the task, I created a test for the `setPrivate` method but it wasn't so trivial because I need authentication for that.

So How do I test a method that runs on the client and the server? I got some errors because `Meteor.userId()` is not present on the server and I was testing from the server. So I made [this change](https://github.com/lnmunhoz/meteor-react-simple-todo/commit/a5a1262e803dbd21e7f80573fd58908f27faebc0#diff-801ce38a8d375a64c138b38e13627b5aR64) to make it works but I'am pretty sure that this is not cool.

I created a [branch on my study repository](https://github.com/lnmunhoz/meteor-react-simple-todo/tree/tests/tasks.setPrivate) to show what I did.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the Meteor 1.3 tutorial context and the `setPrivate` method, then inspect the linked `tests/tasks.setPrivate` branch and referenced commit. Clarify whether the expected outcome is example tests for authenticated client/server methods or removal and redirection to `meteor/todos`; done means the repository direction and testing guidance are unambiguous.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
testing-qa
Issue type
Documentation
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.