Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

Demonstrate #1485, #1487, #2662, #2664 in functional and unit tests #2665

Closed
wants to merge 1 commit into from

Conversation

dougbu
Copy link
Member

@dougbu dougbu commented Jun 8, 2015

  • test additional cases close to these bugs as well

for #1485

  • show odd @Html.CheckBox(), @Html.Hidden() behaviour in unit tests
  • show odd @Html.TextBox() behaviour in functional tests (templates)

for #1487

  • show odd @Html.Value() behaviour in unit tests
  • show odd @Html.RadioButton(), @Html.TextArea() behaviour in functional tests
  • show lack of validation attributes for @Html.RadioButton(), <select> tag helper

for #2662

  • show odd @Html.RadioButton(string.Empty) behaviour in functional tests

for #2664

  • show failures with @Html.ListBox() in unit tests

nits:

  • test IHtmlHelper methods, not extensions
  • use ViewData, not ViewBag in HtmlGeneration_FormController
  • name test methods a bit more consistently
  • rename HtmlHelperValueExtensionsTest to HtmlHelperValueTest

- test additional cases _close_ to these bugs as well

for #1485
- show odd `@Html.CheckBox()`, `@Html.Hidden()` behaviour in unit tests
- show odd `@Html.TextBox()` behaviour in functional tests (templates)

for #1487
- show odd `@Html.Value()` behaviour in unit tests
- show odd `@Html.RadioButton()`, `@Html.TextArea()` behaviour in functional tests
- show lack of validation attributes for `@Html.RadioButton()`, `<select>` tag helper

for #2662
- show odd `@Html.RadioButton(string.Empty)` behaviour in functional tests

for #2664
- show failures with `@Html.ListBox()` in unit tests

nits:
- test `IHtmlHelper` methods, not extensions
- use `ViewData`, not `ViewBag` in `HtmlGeneration_FormController`
- name test methods a bit more consistently
- rename `HtmlHelperValueExtensionsTest` to `HtmlHelperValueTest`
@dougbu
Copy link
Member Author

dougbu commented Jun 8, 2015

/cc @Eilon let's discuss tomorrow and decide if the expectations in the skipped tests match what we want.

@dougbu
Copy link
Member Author

dougbu commented Jun 9, 2015

Will reopen with correct base bracn

@dougbu dougbu closed this Jun 9, 2015
@dougbu dougbu deleted the dougbu/testing.1485.1487.2662.2264 branch June 9, 2015 00:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants