There may not be anything wrong with it; in my experience, breeding a single pair of villagers together takes a long, long time. Of course, since it's based on chance, it might only take a few minutes. I waited like 15 minutes or so—a long time to watch a kettle boil—and they never bred. I put a few more in with them and it went a lot quicker. Five or six will breed somewhat quickly.
You can feed them both manually, too. Just throw carrots or potatoes at their feet. That will help speed the process.
However, I just would bury a set of three doors and surround the villagers with two-block-thick walls of glass. So much easier and far safer. I'd never keep villagers within solid walls due to the five-year-old bug, MC-2025.
Here's a reply I made to someone else here recently:
I've only bred villagers in captivity, which is what I would strongly suggest you do. Once you get as many as you'd like, release them. With only three villagers, in a natural setting, it could take hours for them to breed. They may never do so. Even in captivity, it will take some time.
My suggestion is to corral these three villagers in a small glass bubble (4x4 or so of space) with two-block wide walls. Use glass so they don't glitch into the walls and suffocate. Make it two blocks thick so zombie's don't glitch through it and they don't glitch out. Put a roof on it with a little trap door or two so you can toss down potatoes to feed the villagers since you unfortunately don't have a farmer.
Feed them a lot of potatoes.
Then, very nearby, dig down five blocks below the ground level of the villagers. Next, dig over three blocks. In those three blocks, place doors—all three facing the same way, toward the down shaft your just dug. The last door will be on the down shaft. You can then column up or stair step out however you'd like, but don't cover the block above the exposed door with anything except glass blocks. Leave a hole to open air where the door last stands.
You'll have to feed the villagers, but otherwise, you have an infinite breeder.
(Above) Notice the hole in the floor in the lower-left corner. That's where the three doors are buried. Nothing but glass separates it from the sky. I have my villagers in a potato patch because the farmers will feed them. Clearly, the breeder is overburdened beyond its maximum capacity and I need to cull the herd.
(Above) Looking down at the hole where the doors are buried. You can see the outer door.
(Above) I dug out the side so I could take screenshots. I placed torches to illustrate the number of blocks. Down five, over three.
(Above) Looking up toward the sky. Glass seals the five-block-deep hole.
There are many, many other ways to make an infinite breeder, but this was the quickest and simplest I found. I glossed over and oversimplified all the mechanics, but you can read more about the details on the wiki.
Hope that helps!
Rollback Post to RevisionRollBack
My short story-like journals; quick-and-easy reads: